Skip to content

GROOVY-11658: groovyc: log the command-line arguments at verbose level#2222

Merged
eric-milles merged 1 commit into
masterfrom
GROOVY-11658
May 14, 2025
Merged

GROOVY-11658: groovyc: log the command-line arguments at verbose level#2222
eric-milles merged 1 commit into
masterfrom
GROOVY-11658

Conversation

@eric-milles
Copy link
Copy Markdown
Member

verbose is between info and debug

Copy link
Copy Markdown
Contributor

@perNyfelt perNyfelt left a comment

Choose a reason for hiding this comment

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

Nice solution!

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 13, 2025

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 68.9994%. Comparing base (d51a0b6) to head (53528dd).

Files with missing lines Patch % Lines
...src/main/java/org/codehaus/groovy/ant/Groovyc.java 75.0000% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##               master      #2222        +/-   ##
==================================================
+ Coverage     68.9964%   68.9994%   +0.0030%     
- Complexity      29668      29670         +2     
==================================================
  Files            1423       1423                
  Lines          114332     114330         -2     
  Branches        19835      19835                
==================================================
+ Hits            78885      78887         +2     
+ Misses          28816      28812         -4     
  Partials         6631       6631                
Files with missing lines Coverage Δ
...src/main/java/org/codehaus/groovy/ant/Groovyc.java 48.6956% <75.0000%> (-0.0057%) ⬇️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@paulk-asert
Copy link
Copy Markdown
Contributor

+1

@eric-milles eric-milles merged commit b70de71 into master May 14, 2025
41 checks passed
@eric-milles eric-milles deleted the GROOVY-11658 branch May 14, 2025 13:04
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.

4 participants