Skip to content

AST package updates + Go 1.22#162

Merged
0x19 merged 8 commits intomainfrom
ast-update
Mar 28, 2024
Merged

AST package updates + Go 1.22#162
0x19 merged 8 commits intomainfrom
ast-update

Conversation

@0x19
Copy link
Copy Markdown
Contributor

@0x19 0x19 commented Mar 22, 2024

I did not look for a bit of the time unit tests and updates on of the packages, so... Now I've done it including series of fixes for AST.

FROM NOW ON GO 1.22 IS REQUIRED

This PR additionally resolves:

  • Introducing new Visitor and Storage calculation into the AST
  • Fixing numerous bugs while parsing the contracts...
  • Disabled unit test for observers as that package requires node to be running. As well in the future this package will be removed.
  • Introduces String() into the AST node paths in order to capture actual text of the specific node.
  • Payable conversion reference nil set
  • Client pool unit tests
  • Bumping up the solc-switch to 1.0.4 as 1.0.3 is just bad...

@0x19 0x19 added bug Something isn't working enhancement New feature or request ast Abstract Syntax Tree labels Mar 22, 2024
@0x19 0x19 self-assigned this Mar 22, 2024
@0x19 0x19 changed the title AST package updates AST package updates + Go 1.22 Mar 28, 2024
@0x19 0x19 merged commit a87d757 into main Mar 28, 2024
@0x19 0x19 deleted the ast-update branch March 28, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ast Abstract Syntax Tree bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant