-
Notifications
You must be signed in to change notification settings - Fork 5k
Allow setting --stacktrace with gradle property #18713
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
Conversation
When set to `all`, `summary` or `none`, Gradle will use different warning type display. See <<command_line_interface.adoc#sec:command_line_logging,Command-line logging options>> for details. | ||
_Default is `summary`._ | ||
|
||
`org.gradle.showstacktrace=(internal,all,full)`:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think using the entries from the ShowStacktrace
enum as they are wouldn't look nice here, so I came up with simpler ones.
...ects/logging/src/main/java/org/gradle/internal/logging/LoggingConfigurationBuildOptions.java
Outdated
Show resolved
Hide resolved
subprojects/docs/src/docs/userguide/running-builds/build_environment.adoc
Show resolved
Hide resolved
Looks good to me, added just some suggestions to consider. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of questions/comments
...ects/logging/src/main/java/org/gradle/internal/logging/LoggingConfigurationBuildOptions.java
Show resolved
Hide resolved
subprojects/launcher/src/integTest/groovy/org/gradle/launcher/StacktraceIntegrationTest.groovy
Outdated
Show resolved
Hide resolved
subprojects/launcher/src/integTest/groovy/org/gradle/launcher/StacktraceIntegrationTest.groovy
Outdated
Show resolved
Hide resolved
...ects/logging/src/main/java/org/gradle/internal/logging/LoggingConfigurationBuildOptions.java
Show resolved
Hide resolved
...ects/logging/src/main/java/org/gradle/internal/logging/LoggingConfigurationBuildOptions.java
Outdated
Show resolved
Hide resolved
6dc14e3
to
1a44dc5
Compare
@big-guy ptal |
@bot-gradle test rFM |
OK, I've already triggered ReadyForMerge build for you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, thought I had already +1
@bot-gradle test and merge |
Your PR is queued. See the queue page for details. |
OK, I've already triggered a build for you. |
In 8.x this property is still there, but the documentation is missing #30278 |
Fixes #13708