-
Notifications
You must be signed in to change notification settings - Fork 5k
Adds a command line argument which can be used to suppress all extra help messages if the build should fail with an exception #21033
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
…help messages if the build should fail with an exception
@bot-gradle test QFL |
OK, I've already triggered the following builds for you: |
I don't think we should do this, but I agree that the current output could be more useful. I'd rather make the output more useful in some small way over turning it off entirely. |
This pull request has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be closed if no further activity occurs. If you intend to work on this pull request, please ask the team to reopen the PR or push a new PR. Thank you for your contributions. |
For what it's worth Gradle's own help output contains this:
I don't see how this annoying and repetetive help message qualifies as an "error". I would rather prefer that Tested with Gradle 8.4. |
One thing to note here is that there's an inconsistency in the build output when I'm leaning towards implementing the proposed behavior. At the same time, we currently don't have the capacity to work on this. Would anyone be interested in creating a pull request? |
Related: #28373. I think we should try to get something done IRT this. There seems to be a good amount of sustained interest. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. If you don't want the stale bot to close it, then set a milestone for it. |
@donat - You closed the linked issue as Not Planned. Should this be closed, too? |
Fixes #20964