-
Notifications
You must be signed in to change notification settings - Fork 4.6k
[Do not Merge] Cms New bot testing #49793
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
|
A new Pull Request was created by raoatifshad for master. It involves the following packages:
Dr15Jones, makortel, smuzaffar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49793/47426
|
|
code-checks |
|
please test |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49793/47427 |
|
-1 Failed Tests: RelVals Failed RelVals |
|
test parameters:
|
|
@cmsbuild explain test parameters |
|
@cmsbuild explain status |
I will investigate this. |
|
@cmsbuild please +1 |
|
Can I call @cmsbuild please +1 like that |
|
hold unhold |
Yes. It is not an official syntax, but a side-effect of prefix removal (and it also works now with old bot since #2551). @smuzaffar let me know if this is not a desired behavior. |
|
test parameters:
|
Fixed. |
|
please hold |
|
Pull request has been put on hold by raoatifshad |
|
please unhold |
|
@cmsbuild please +1 and also test workflows 1.0 |
|
can I unhold |
|
Pull request has been put on hold by raoatifshad |
|
I think we should hold |
|
I think we should |
|
hold |
|
Pull request has been put on hold by raoatifshad |
|
may I unhold |
|
can I unhold |
Fixed. |
PR description: