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

maven-compiler-plugin maintenance #1731

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

dotasek
Copy link
Collaborator

@dotasek dotasek commented Aug 30, 2024

Updates the maven-compiler plugin.

Adds additional memory flags to prevent excessive memory usage on CI runs.

Sorts out the previous inheritance of HAPI's compiler settings.

Removes the deprecated forceJavacCompilerUse configuration, fixing a previous warning.

and stop using deprecated forceJavacCompilerUse
Copy link

codecov bot commented Aug 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 12.65%. Comparing base (07f1981) to head (d7c36b1).
Report is 7 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1731      +/-   ##
============================================
- Coverage     12.65%   12.65%   -0.01%     
+ Complexity    32720    32718       -2     
============================================
  Files          2204     2204              
  Lines        674167   674167              
  Branches     198746   198746              
============================================
- Hits          85340    85339       -1     
- Misses       558640   558641       +1     
  Partials      30187    30187              

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

@dotasek dotasek changed the title Update maven-compiler-plugin maven-compiler-plugin maintenance Sep 13, 2024
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