Skip to content

Fix missed generated code from PatternExpr refactor#4414

Merged
jlerbsc merged 2 commits intojavaparser:masterfrom
johannescoetzee:johannes/fix-codegen
May 6, 2024
Merged

Fix missed generated code from PatternExpr refactor#4414
jlerbsc merged 2 commits intojavaparser:masterfrom
johannescoetzee:johannes/fix-codegen

Conversation

@johannescoetzee
Copy link
Copy Markdown
Collaborator

@johannescoetzee johannescoetzee commented May 6, 2024

While working on the RecordPatternExpr feature, I noticed that I missed some generated code while filtering style changes from actual changes in #4387. For this PR, I just re-ran the core metamodel generator and core generators again and added the relevant changes.

I would like to draw attention to my reformatting proposal (see #4408) as well. I understand that reformatting the entire project is not a decision to be taken lightly, but I think it is important to prevent mistakes like these while also making contributions easier. The approach I mentioned in the issue is of course just one approach and I'm open to any suggestions on how to improve it. Even if that approach isn't chosen, I hope that my ideas on how to minimise the impact of such a change are of some use.

@jlerbsc jlerbsc merged commit 73d9c4f into javaparser:master May 6, 2024
@jlerbsc jlerbsc added this to the next release milestone May 6, 2024
@jlerbsc jlerbsc added PR: Added A PR that introduces new behaviour (e.g. functionality, tests) PR: Fixed A PR that offers a fix or correction and removed PR: Added A PR that introduces new behaviour (e.g. functionality, tests) labels May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Fixed A PR that offers a fix or correction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants