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
Static Analyzers: Add checker for use of edm::ParameterSet::exists or ::existsAs<T>. #36424
Static Analyzers: Add checker for use of edm::ParameterSet::exists or ::existsAs<T>. #36424
Conversation
…ring instead of beginning in case const at beginning.
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36424/27242
Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
Rather than Deprecated, I would maybe place this check under "CMS code rules" (we recommend to avoid them, even if we don't mention them explicitly in the code rules). |
@cmsbuild, please test |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36424/27244
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-36424/27246
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36424/27273
|
Pull request #36424 was updated. @cmsbuild, @smuzaffar, @Dr15Jones, @makortel can you please check and sign again. |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ec95de/21138/summary.html 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, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Tested on PhysicsTools/PatAlgos which makes use of both. Examples of test output to console: