Skip to content

Upgrade Dropwizard BOM from 1.0.5 to 4.0.7#35

Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1776966571-upgrade-dropwizard-4x
Open

Upgrade Dropwizard BOM from 1.0.5 to 4.0.7#35
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1776966571-upgrade-dropwizard-4x

Conversation

@devin-ai-integration
Copy link
Copy Markdown

Summary

Decouples the Dropwizard BOM version from ${project.version} and upgrades it to 4.0.7.

  • Adds a <properties> section with <dropwizard.version>4.0.7</dropwizard.version>
  • Changes the BOM <version> from ${project.version} (1.0.5) to ${dropwizard.version} (4.0.7)
  • Project version remains 1.0.5 (unchanged)

This PR only changes pom.xml. The javax → jakarta source code migration required by Dropwizard 4.x will be handled in a separate follow-up PR. The project will not compile until that migration is complete.

Review & Testing Checklist for Human

  • Confirm all Dropwizard artifacts used in the POM (dropwizard-core, dropwizard-auth, dropwizard-assets, dropwizard-http2, dropwizard-hibernate, dropwizard-migrations, dropwizard-views-freemarker, dropwizard-views-mustache, dropwizard-metrics-graphite, dropwizard-testing) exist in version 4.0.7
  • Verify <version>1.0.5</version> on line 7 is unchanged
  • Acknowledge that CI/build will fail until the javax→jakarta migration PR lands on top of this

Notes

  • Previously, the BOM version was ${project.version}, which only worked because the project version (1.0.5) coincidentally matched the intended Dropwizard version. This PR makes the dependency explicit.
  • Test plan: After the follow-up javax→jakarta migration PR is merged, run mvn clean verify to confirm the full build passes.

Link to Devin session: https://app.devin.ai/sessions/ce3e64a3e3ff40cd8cd78e913ded1939
Requested by: @WesternConcrete

- Add <properties> section with <dropwizard.version>4.0.7</dropwizard.version>
- Update BOM import to use ${dropwizard.version} instead of ${project.version}
- Keep project version as 1.0.5

Co-Authored-By: Wes Convery <2wconvery@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant