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
Running code-format for core #28891
Running code-format for core #28891
Conversation
please test |
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28891/13669
|
The tests are being triggered in jenkins. |
A new Pull Request was created by @cmsbuild for master. It involves the following packages: DataFormats/Common @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Were these files missed before, or did we get an update in clang-format? |
I think it is bit of both. Clang format/tidy were not run for those packages for which there were already open PRs and we also have new llvm ( 9.0.1) now in 11.1.X. |
At least for these 3 files, the immediately preceding commit in the history is the result of running clang format, when I did it back in May. So something must have changed in clang-format or the way we are running it. |
yes, looks like side effect of new llvm version 9.0.1. Last time when we run clang-format was with llvm 7. |
+1 |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
+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. @davidlange6, @silviodonato, @fabiocos (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Applying code-format for CMSSW category core.
See the build logs here https://cmssdt.cern.ch/jenkins/job/GitHub-refactor-cmssw-module/448//console
cms-bot has successfully run the following: