Skip to content

[apex] Summit-AST Apex module - Part 3 - initializers#4171

Merged
adangel merged 7 commits into
pmd:experimental-apex-parserfrom
aaronhurst-google:experimental-apex-parser
Nov 10, 2022
Merged

[apex] Summit-AST Apex module - Part 3 - initializers#4171
adangel merged 7 commits into
pmd:experimental-apex-parserfrom
aaronhurst-google:experimental-apex-parser

Conversation

@aaronhurst-google

@aaronhurst-google aaronhurst-google commented Oct 20, 2022

Copy link
Copy Markdown
Contributor

Build and translate nodes related to variable declaration initializers.

Ref #3766

eklimo and others added 7 commits October 17, 2022 03:23
Change-Id: I6c4bc34a85e80302da8ef4043a65b7cbec426ee1
Make AST node constructors package-private.
Cleanup imports.
Change-Id: Ibaef3a59307633647129b4971a382ee85a5c0d53
@adangel adangel changed the title Summit-AST Apex module - Part 3 - initializers [apex] Summit-AST Apex module - Part 3 - initializers Oct 25, 2022
@aaronhurst-google

Copy link
Copy Markdown
Contributor Author

Hi @adangel... to give you some forward visibility. I've got the following changes lined up:

  • Part 4 - statements
  • Part 5 - annotations. (This completes the skeleton of the translation. All the node types are at least partially handled.)
  • Some test and TODO fixes from @eklimo

After that, I'll be continuing to burn down the test suite failures and TODOs.

@adangel adangel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, LGTM!

I'll merge it into experimental-apex-parser.

@adangel adangel merged commit b75c700 into pmd:experimental-apex-parser Nov 10, 2022
@adangel adangel added this to the 7.0.0 milestone Feb 13, 2024
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.

3 participants