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
Moved plugins to FWCore/Integration/plugins #40448
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40448/33598
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages:
@cmsbuild, @smuzaffar, @Dr15Jones, @makortel can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test TestIntegrationParameterSet had ERRORS Comparison SummarySummary:
|
87a7b22
to
bfde062
Compare
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40448/33619
|
Pull request #40448 was updated. @makortel, @smuzaffar, @Dr15Jones can you please check and sign again. |
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-40448/33638
|
Pull request #40448 was updated. @makortel, @smuzaffar, @Dr15Jones can you please check and sign again. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9d562f/29882/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 SummarySummary:
|
+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. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Someone needs to fix a unit test related to a moved file. I guess the comment is also incorrect :) ... I'll do it unless someone else is already working on it... https://cmssdt.cern.ch/dxr/CMSSW/source/FWCore/Integration/test/ProducerWithPSetDesc.cc#306
|
@wddgit I don't think anyone is working on that, so feel free to go ahead. |
Hello @Dr15Jones, @wddgit |
PR description:
These were previously in the test directory. Also consolidated source files into fewer plugin shared libraries being made.
PR validation:
Code compiles and tests succeed.