Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Always build source jar #1037

Merged
merged 1 commit into from
Aug 4, 2024
Merged

Always build source jar #1037

merged 1 commit into from
Aug 4, 2024

Conversation

philsttr
Copy link
Collaborator

@philsttr philsttr commented Aug 4, 2024

There was a regression introduced in 8.0 when building the source jar during releases when the maven-release-plugin was upgraded.

To simplify things, and avoid that in the future, I changed the build to always package the source jar.

Fixes #1036

There was a regression introduced when building the source jar during releases when the maven-release-plugin was upgraded.

To simplify things, and avoid that in the future, I changed the build to alwas package the source jar.

Fixes #1036
Copy link

codecov bot commented Aug 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.82%. Comparing base (1ac92af) to head (3898d32).
Report is 13 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1037      +/-   ##
============================================
+ Coverage     71.78%   71.82%   +0.03%     
- Complexity     1417     1429      +12     
============================================
  Files           177      178       +1     
  Lines          5178     5199      +21     
  Branches        534      539       +5     
============================================
+ Hits           3717     3734      +17     
+ Misses         1197     1195       -2     
- Partials        264      270       +6     

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

@philsttr philsttr merged commit a8e6b81 into main Aug 4, 2024
12 checks passed
@philsttr philsttr deleted the enable_source branch August 4, 2024 18:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release 8.0 source artifact in Maven Central does not contain the source code
1 participant