-
Notifications
You must be signed in to change notification settings - Fork 79
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
Evaluate the quality-monitor github action #1733
Comments
It's worth trying, maybe in PDE first? |
please not. we have enough trouble with the jenkins plugins. |
I don't think this is nice commentary. The title here is to evaluate something. Surely its reasonable to evaluate something. I see both you and @laeubi doing so very much work to make things better. Sure, sometimes there are growing pains, sometimes there are differences of opinion, and sometimes some things appear worse than before. For example, the warning elimination effort is great, but already there have been two regressions as a result, one of which cost me the better part of a day to fix. But I don't say, "eliminate warnings, please don't!", because I respect that you are doing very good things in general with the best of intentions. Of course none of us is perfect, so mistakes can and do happen to the best of us. I think it's important to remain respectful and to recognize the good work that all the other people are doing. It's not as if @laeubi and I always see eye-to-eye, but I greatly respect his huge contribution to releng improvements, often with complaints as his only thank you. It would be good to be just a little nicer. |
@laeubi Do you plan to do resume this one or it should be closed? |
Yes I'll probably pick that up later. |
See:
This is similar to what Jenkins already offers but gives a much more integrated/nicer feedback and might even replace the Junit result panel we currently have.
The text was updated successfully, but these errors were encountered: