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

Change to new jetty maven plugin #5835

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dotasek
Copy link
Contributor

@dotasek dotasek commented Apr 9, 2024

jetty-maven-plugin is not available for Jetty beyond version 11.x, which can result in errors in downstream projects due to an inability to resolve the 12+ version we're currently using.

It has been changed to the following:

<groupId>org.eclipse.jetty.ee10</groupId>
<artifactId>jetty-ee10-maven-plugin</artifactId>

See here for details: https://github.com/jetty/jetty.project/pull/10939/files#diff-a4f36866106c36c5a5f6342ad8343f2f04c930d48171f484deead1e2444ebcc7

Copy link

github-actions bot commented Apr 9, 2024

Formatting check succeeded!

Copy link

codecov bot commented Apr 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.33%. Comparing base (497b9f2) to head (2df567a).
Report is 33 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5835      +/-   ##
============================================
- Coverage     83.39%   83.33%   -0.06%     
- Complexity    26927    26948      +21     
============================================
  Files          1681     1686       +5     
  Lines        103965   104134     +169     
  Branches      13189    13218      +29     
============================================
+ Hits          86702    86784      +82     
- Misses        11613    11702      +89     
+ Partials       5650     5648       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

None yet

2 participants