-
Notifications
You must be signed in to change notification settings - Fork 196
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
Remove the message about jenkinsci-users ML #97
Conversation
Asking people to simply send this to the ML has FWICT never led to any bugfix. This generally is just sent as is with no details. Better ask people to report issues to avoid spamming the whole users ML.
The one I was considering changing was the "oops" page, but when I reviewed it, the messages actually made sense. It's possible that I was thinking of this message and just didn't remember it was distinct from 'oops'. |
Link to https://wiki.jenkins-ci.org/display/JENKINS/How+to+report+an+issue or its clone on Jenkins.io would be useful |
@@ -924,7 +924,7 @@ protected Result doRun(final BuildListener listener) throws Exception { | |||
return Result.FAILURE; | |||
} catch (RuntimeException e) { | |||
// bug in the code. | |||
e.printStackTrace(listener.error("Processing failed due to a bug in the code. Please report this to [email protected]")); | |||
e.printStackTrace(listener.error("Processing failed due to a bug in the code. Please report this to the issue tracker")); |
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.
Or simply
e.printStackTrace(listener.error("Processing failed unexpectedly"));
If the user is motivated enough to capture details and steps to reproduce and so on, they will figure out how to use the issue tracker. Given that we are swamped by vague and unreproducible bugs as it is I am reluctant to make it sound simpler than it is.
|
@batmat can you add the link to https://jenkins.io/redirect/report-an-issue as advised by others ? |
@aheritier Thanks for the fix, sorry for lagging those days. Seems like there's enough approval and you can merge this? Thanks |
Asking people to simply send this to the ML has FWICT never led to any bugfix. This generally is just sent as is with no details. Better ask people to report issues to avoid spamming the whole users ML.
Last example this morning pushed me to finally propose this PR.
@daniel-beck I think you already talked about removing this in the past.