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

Update jetty Docker tag to v12 #290

Closed
wants to merge 1 commit into from
Closed

Update jetty Docker tag to v12 #290

wants to merge 1 commit into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Aug 15, 2023

Mend Renovate

This PR contains the following updates:

Package Type Update Change
jetty final major 10.0.16-jre17 -> 12.0.1-jre17

⚠ Dependency Lookup Warnings ⚠

Warnings were logged while processing this repo. Please check the Dependency Dashboard for more information.


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot force-pushed the renovate/jetty-12.x branch from ff48b68 to 362bc5e Compare September 13, 2023 01:04
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

@timja @lemeurherve @NotMyFault do you agree that we could upgrade to this major version 12?

If yes, then it supersedes #300

@NotMyFault
Copy link
Member

Can we actually do that? We consume jetty 11 or 10 in the build script still

@dduportal
Copy link
Contributor

Can we actually do that? We consume jetty 11 or 10 in the build script still

Oh, did not know that. Then I guess it requires more work (my knowledge of the internals of this app are close to zero so I prefer asking ;) )

@NotMyFault
Copy link
Member

Yeah, the blocker - last time I checked - was that Jetty 12 moves from EE to Jakarta bindings while stapler (or another core dependency this repo uses) makes use of EE still and doesn't offer both for a migration.

Likely, we need to wait until we move on to Jakarta bindings until we can move on here too.

@NotMyFault
Copy link
Member

Might be worth to exclude that from Dependabot for now to keep noise calm

@renovate renovate bot force-pushed the renovate/jetty-12.x branch from 362bc5e to 476cd60 Compare September 13, 2023 11:06
@lemeurherve
Copy link
Member

🔕 Ignore: Close this PR and you won't be reminded about this update again.

@renovate
Copy link
Contributor Author

renovate bot commented Sep 15, 2023

Renovate Ignore Notification

Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future 12.x releases. But if you manually upgrade to 12.x then Renovate will re-enable minor and patch updates automatically.

If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.

@renovate renovate bot deleted the renovate/jetty-12.x branch September 15, 2023 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants