Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Core semantics #5

Merged
merged 7 commits into from
Aug 19, 2024
Merged

Core semantics #5

merged 7 commits into from
Aug 19, 2024

Conversation

dwightguth
Copy link
Contributor

This PR adds the core elements of the semantics needed to begin the feature-implementation loop.

It includes:

  • Cleanup to the Makefile
  • A basic configuration declaration and some basic internal sorts
  • Code to handle processing of interface definitions
  • A simple, as-yet-untested script for running the semantics
  • Some minor grammar tweaks
  • Adding the entire forge test suite into the example file for Uniswap so it can be more efficiently run and tested.

My next task is going to be to populate the backlog with a prioritized list of tasks for filling out the features needed to run this test suite.

@dwightguth dwightguth marked this pull request as ready for review August 16, 2024 19:18
Copy link
Contributor

@mariaKt mariaKt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

For the test harness, it looks good to me, but I will need to get more comfortable with Solidity and the specific contracts to fully understand.

@dwightguth dwightguth merged commit 6afd4fe into main Aug 19, 2024
1 check passed
@dwightguth dwightguth deleted the core-semantics branch August 19, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants