-
Notifications
You must be signed in to change notification settings - Fork 1
Bump cpp-linter/clang-tools-static-binaries to master-11dfa474 #118
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #118 +/- ##
=======================================
Coverage 95.42% 95.42%
=======================================
Files 7 7
Lines 284 284
=======================================
Hits 271 271
Misses 13 13 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the clang-tools-static-binaries reference and associated checksum files, and extends the CI workflow to cover additional paths and a new Python version.
- Bump
release_tag
tomaster-2200605c
- Update SHA512 sums for
clang-format-12
binaries on Windows, macOS, and Linux - Add
clang_tools/*
andtests/*
to PR triggers and include Python 20 in the test matrix
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
tests/clang-format-12_windows-amd64.sha512sum | Updated Windows AMD64 SHA512 checksum for clang-format-12 |
tests/clang-format-12_macosx-amd64.sha512sum | Updated macOS AMD64 SHA512 checksum for clang-format-12 |
tests/clang-format-12_linux-amd64.sha512sum | Updated Linux AMD64 SHA512 checksum for clang-format-12 |
clang_tools/init.py | Bumped release_tag to new commit identifier |
.github/workflows/python-test.yml | Added additional watched paths and extended Python versions list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Bumps the cpp-linter/clang-tools-static-binaries
release tag to master-11dfa474
, updates associated SHA512 checksum files, and expands CI to cover the updated paths and a new tool version.
- Update SHA512 checksums for clang-format-12 binaries on Windows, macOS, and Linux
- Bump
release_tag
inclang_tools/__init__.py
to the new commit - Extend GitHub Actions to include
clang_tools/*
,tests/*
, and add version 20/macos-13 to the test matrix
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
tests/clang-format-12_windows-amd64.sha512sum | Update SHA512 checksum for Windows binary |
tests/clang-format-12_macosx-amd64.sha512sum | Update SHA512 checksum for macOS binary |
tests/clang-format-12_linux-amd64.sha512sum | Update SHA512 checksum for Linux binary |
clang_tools/init.py | Bump release_tag to master-11dfa474 |
.github/workflows/python-test.yml | Add new paths to trigger CI and bump matrix |
Comments suppressed due to low confidence (4)
tests/clang-format-12_windows-amd64.sha512sum:1
- The leading
1
before the checksum appears to be accidental and will break the SHA512 validation. Remove the1
prefix so the line starts directly with the checksum.
1 09d379023eb9faacfe08c2254440268bdc0249956ddd38b26e59230abc7b514390f279973ab6c0a069ae79504422884808fee2cf617741c56729e7b6b5fd4fab *clang-format-12_windows-amd64
tests/clang-format-12_macosx-amd64.sha512sum:1
- The leading
1
before the checksum appears to be accidental and will break the SHA512 validation. Remove the1
prefix so the line starts directly with the checksum.
1 b1aa513baf640836bcb7dc02f6fe348c5a0d42e8fd1836e4019147141e807a311f64e00eaeb4031f8077eb39a019aca6bcc2137b5af274644b5769d868e16fe5 clang-format-12_macosx-amd64
tests/clang-format-12_linux-amd64.sha512sum:1
- The leading
1
before the checksum appears to be accidental and will break the SHA512 validation. Remove the1
prefix so the line starts directly with the checksum.
1 79afa99481f071bee2be3e72d2b1641b60427cc202360e44cf4e5dc6f977c4e4223e5a29430a15c9c80720dd6f75801f4811d2848ee52fe014fc232a99a589af clang-format-12_linux-amd64
.github/workflows/python-test.yml:80
- The exclusion for
version: 19, os: macos-latest
was removed despite the known issue in upstream (Support clang-tools version 15 #40). Consider re-introducing the exclusion until that bug is resolved to avoid CI failures.
version: [ 8, 9, 10, 11, 12, 12.0.1, 13, 14, 15, 16, 17, 18, 19, 20 ]
|
Bump cpp-linter/clang-tools-static-binaries to
master-11dfa474
closes #111 #116