Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

restore-moditect-pom.xml restore plugin #929

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

stleary
Copy link
Owner

@stleary stleary commented Jan 7, 2025

What problem does this code solve?
Restores 20240303 behavior - module-info

Does the code still compile with Java6?
Yes

Risks
Low - this is a restore from a previous release

Changes to the API?
No

Will this require a new release?
YES - 20241224 does not contain the module

Should the documentation be updated?
No

Does it break the unit tests?
No

Was any code refactored in this commit?
No

Review status
APPROVED - by myself.
No comment window, for immediate release

@stleary stleary merged commit 324090a into master Jan 7, 2025
12 checks passed
@@ -193,6 +193,30 @@
</execution>
</executions>
</plugin>
<plugin>
Copy link
Contributor

Choose a reason for hiding this comment

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

@stleary
Copy link
Owner Author

stleary commented Jan 8, 2025

@michael-ameri Thanks for catching this.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants