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.

Sign up for GitHub

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

Jump to bottom

AST Reference Resolver Improvements #48

Merged
merged 5 commits into from
Aug 4, 2023
Merged

AST Reference Resolver Improvements #48

merged 5 commits into from
Aug 4, 2023

Conversation

0x19
Copy link
Contributor

@0x19 0x19 commented Aug 4, 2023 •
edited
Loading

Few quite nice additions:

  • Most of the references, forward or not forward statement discovery are resolved. Some are replaced with unknown and will be fixed in the future.
  • Automatic Source Detection is now in place at least for openzeppelin contracts. It's easy to extend by applying few rules. This process will be updated in the future.
  • New AST expressions IndexRange and FunctionCallOption.
  • Reference unit testing.
  • Sources unit testing.

@0x19 0x19 added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request ast Abstract Syntax Tree high priority High Priority Tasks labels Aug 4, 2023
@0x19 0x19 self-assigned this Aug 4, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5761471485

  • 7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 77.186%
Totals Coverage Status
Change from base Build 5761320576: 0.03%
Covered Lines: 6188
Relevant Lines: 8017
💛 - Coveralls

@0x19 0x19 merged commit 700fca4 into main Aug 4, 2023
@0x19 0x19 deleted the ast/reference-resolver branch August 4, 2023 22:19
@0x19 0x19 mentioned this pull request Aug 6, 2023
Closed
@0x19 0x19 linked an issue Aug 6, 2023 that may be closed by this pull request
Abstract Syntax Tree (AST) Forward Statement Discovery #44
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@0x19 0x19

Labels
ast Abstract Syntax Tree bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request high priority High Priority Tasks
Projects
None yet
Milestone
Abstract Syntax Tree (AST) package
Development

Successfully merging this pull request may close these issues.

Abstract Syntax Tree (AST) Forward Statement Discovery
2 participants
@0x19 @coveralls