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

Jetty 10.0.x Dependency Cleanup #6893

Merged
merged 5 commits into from
Sep 22, 2021
Merged

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Sep 22, 2021

A pass through the poms in jetty-10.0.x with the following in mind.

  • All plugins have property versions
  • All dependencies have property versions
  • All dependencies are declared in top level pom
  • All maven sub-modules use version-less dependencies (relying on version from top level pom's dependencyManagement)
  • All old artifacts on javax.* names use new jakarta.* names (this is just the artifacts, not the code) - still using Jakarta EE 8 rules for javax.* namespace in code.
  • Proper exclusion of javax.* artifacts where they are being pulled in transitively

+ All plugins and dependencies are alphabetized now
+ All plugins and dependency versions have properties now

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime self-assigned this Sep 22, 2021
@joakime joakime added Build dependencies Pull requests that update a dependency file labels Sep 22, 2021
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime merged commit 66b9428 into jetty-10.0.x Sep 22, 2021
@joakime joakime deleted the jetty-10.0.x-reorg-dependencies branch September 22, 2021 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants