-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
ME1a CLCT will use ME1a SimHit Strip and be saved into ME1a plot #37319
Conversation
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37319/28959 ERROR: Unable to merge PR. See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37319/28959/cms-checkout-topic.log |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37319/28960
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37319/28963
|
A new Pull Request was created by @sifuluo (Sifu Luo) for master. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @pbo0, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Thank you @sifuluo! As additional (unrelated but welcome) cleaning, could you also please:
with
and remove the doing-nothing destructor from CSCStubResolutionValidation.cc |
Either/both the changes to the destructor will fail the compiling. |
please test |
@perrotta the test didn't seem to be started |
Indeed: bot seems stuck |
humm, strange , I am am checking |
please test |
code-checks |
@perrotta , looks like github again has issues today ( https://www.githubstatus.com/ ) . That is why bot was not able to add the code-checks status. lets wait for github to come to normal state |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37319/28966
|
Pull request #37319 was updated. @emanueleusai, @ahmad3213, @jfernan2, @rvenditti, @pbo0, @pmandrik can you please check and sign again. |
looks like github is coming back, bot has reacted and started the test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-d51e85/23328/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see more details here: Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+1 |
@sifuluo please create backports of this PR to propagate the fix |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Trying to fix the comparison failures mentioned: #37268
PR validation: