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

Pass MavenExecutionContextView into MavenParser #629

Closed

timtebeek wants to merge 1 commit into main from pass_mavenExecutionContext_into_parser

Contributor

timtebeek commented Sep 25, 2023

This view containing the Maven settings above is only created, not used or passed down.


          Pass MavenExecutionContextView into MavenParser

6742b4a

timtebeek self-assigned this

Contributor Author

timtebeek commented Sep 25, 2023

We call .putMessage on the delegate all the same, so looks like we don't have to pass this down. It's puzzling a bit, but that's about it.

timtebeek closed this

timtebeek deleted the pass_mavenExecutionContext_into_parser branch

September 25, 2023 18:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet