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

Rename otp-shaded-jar artifact and fix deployment to Maven Central #6331

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

t2gran
Copy link
Member

@t2gran t2gran commented Dec 10, 2024

Summary

Add back the shaded jar as an Maven artifact.

NOTE! The new name is otp-shaded-<version>.jar (not otp-<version>-shaded.jar)

This will keep adding the "one-jar" as an artifact to Maven Central for the next release. The doc
is updated. I updated to the next version number where apropriate, since the path in Maven Central
will change. Therefore, the new path does not exist yet - it will exist when the next release (2.7.0) is made.

Issue

This is related to #6267

Unit tests

🟥 Not relevant.

Documentation

✅ All scripts and documentation referencing the shaded jar is updated.

Changelog

✅ This is relevant for everyone maintaining OTP deployments - so to "spreed the news" we will to include this
in the chagelog - even if it is not relevant to product owners.

Bumping the serialization version id

🟥 Not needed.

The removed section was commented out by @abyrd in May 2014, and replaced later the same year
with the current filter configuration (see filter config above the removed code).
@t2gran t2gran added this to the 2.7 (next release) milestone Dec 10, 2024
@t2gran t2gran requested a review from a team as a code owner December 10, 2024 19:44
Copy link

codecov bot commented Dec 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.82%. Comparing base (0efdb4f) to head (f85cf52).
Report is 97 commits behind head on dev-2.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #6331      +/-   ##
=============================================
+ Coverage      69.79%   69.82%   +0.02%     
- Complexity     17787    17829      +42     
=============================================
  Files           2017     2020       +3     
  Lines          76035    76269     +234     
  Branches        7781     7806      +25     
=============================================
+ Hits           53072    53254     +182     
- Misses         20260    20301      +41     
- Partials        2703     2714      +11     

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

@t2gran t2gran requested a review from vpaturet December 11, 2024 08:50
@t2gran t2gran merged commit d936e6b into opentripplanner:dev-2.x Dec 13, 2024
5 checks passed
@t2gran t2gran deleted the add_shaded_jar_artifact branch December 13, 2024 12:44
t2gran pushed a commit that referenced this pull request Dec 13, 2024
@t2gran t2gran changed the title Add shaded jar artifact Rename otp-shaded-jar artifact and fix deployment to Maven Central Dec 13, 2024
t2gran added a commit that referenced this pull request Dec 13, 2024
…ifact and fix deployment to Maven Central
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