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.

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

Minor NBM wizard pom generation fixes and plugin version bumps #7628

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

mbien
Copy link
Member

@mbien mbien commented Aug 1, 2024

While testing nbm-maven-plugin 14.2, I noticed that the wizard would generate redundant compiler plugins for child poms and it also missed out on some version bumps.

This reminded me that the nbm-archetype isn't used for anything right now (but this won't fix this).

To test this, nbm-maven-plugin v14.2 and netbeans-platform-app-archetype v1.24 should be put into the local repo first since they aren't released yet.

@mbien mbien added Maven [ci] enable "build tools" tests Platform [ci] enable platform tests (platform/*) ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Aug 1, 2024
@mbien mbien added this to the NB23 milestone Aug 1, 2024
@mbien mbien marked this pull request as draft August 1, 2024 15:43
@mbien
Copy link
Member Author

mbien commented Aug 4, 2024

I hope CI will pass once nbm-maven-plugin 14.2 is released

edit: fixed now + 14.2 is available

 - maven wizard iterator needed some updates to work better with the
   new archetypes
 - bumped the version fallbacks in MavenVersionSettings
@mbien mbien force-pushed the nbm-wizard-version-fixes_delivery branch from 1669a89 to 4f80c4d Compare August 5, 2024 14:07
@mbien mbien marked this pull request as ready for review August 5, 2024 15:32
Copy link
Contributor

@ebarboni ebarboni left a comment

Choose a reason for hiding this comment

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

LGTM

@ebarboni ebarboni merged commit be2a72c into apache:delivery Aug 7, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Maven [ci] enable "build tools" tests Platform [ci] enable platform tests (platform/*)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants