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.

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

[UNDERTOW-2091] spotbugs-maven-plugin needs to be updated to work with JDK17 #1317

Merged
merged 1 commit into from
May 31, 2022
Merged

[UNDERTOW-2091] spotbugs-maven-plugin needs to be updated to work with JDK17 #1317

merged 1 commit into from
May 31, 2022

Conversation

jstourac
Copy link
Contributor

@jstourac
Copy link
Contributor Author

Note: the version of the spotbugs-maven-plugin used in this PR requires Apache Maven 3.3.9+ (restriction added in https://github.com/spotbugs/spotbugs-maven-plugin/releases/tag/spotbugs-maven-plugin-4.4.2.1). Since Undertow define minimum Maven version via it's parent pom, I have also created a following issue jboss/jboss-parent-pom#130

@fl4via fl4via added enhancement Enhances existing behaviour or code next release This PR will be merged before next release or has already been merged (for payload double check) under verification Currently being verified (running tests, reviewing) before posting a review to contributor waiting CI check Ready to be merged but waiting for CI check and removed under verification Currently being verified (running tests, reviewing) before posting a review to contributor waiting CI check Ready to be merged but waiting for CI check labels May 31, 2022
@fl4via fl4via merged commit 7a20f52 into undertow-io:master May 31, 2022
@fl4via fl4via added the dependency upgrade Pull requests that update a dependency file label May 31, 2022
@jstourac jstourac deleted the spotbugsJdk17 branch May 31, 2022 22:23
@fl4via fl4via removed the next release This PR will be merged before next release or has already been merged (for payload double check) label Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependency upgrade Pull requests that update a dependency file enhancement Enhances existing behaviour or code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jstourac @fl4via