-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Phase2 inner tracker class def xml fix #47168
Phase2 inner tracker class def xml fix #47168
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-47168/43404 |
A new Pull Request was created by @sihyunjeon for master. It involves the following packages:
@Moanwar, @cmsbuild, @jfernan2, @mandrenguyen, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Size: This PR adds an extra 20KB to repository Comparison SummarySummary:
|
+Upgrade |
+1 |
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. @sextonkennedy, @mandrenguyen, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
what changes are expected in the output if any,
Bug fix so that QCore -related objects can actually be stored in the EDM output
what other PRs or externals it depends upon if any,
quarter core processor for HL-LHC inner tracker developments #45759 following up with lingering comments from this PR. proper class definitions in xml file (checked the output actually stores QCore objects)