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
Prepare for ASCII copyrights by using multiple license files in pre-commit script #12693
Conversation
⏱️ 6h 36m total CI duration on this PR
🚨 2 jobs on the last run were significantly faster/slower than expected
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12693 +/- ##
=========================================
- Coverage 62.6% 62.6% -0.1%
=========================================
Files 823 822 -1
Lines 184343 184199 -144
=========================================
- Hits 115499 115390 -109
+ Misses 68844 68809 -35 ☔ View full report in Codecov by Sentry. |
700b987
to
b9a72ff
Compare
rebased, and tweaked a little. |
377c0d6
to
48eb809
Compare
48eb809
to
97cc3fd
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
13dfda4
to
aed6590
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ Forge suite
|
✅ Forge suite
|
Description
Add multiple allowed license files to pre-commit script, tolerating
existing UTF8-encoded (c) symbols but adding ASCII-only notices if
missing. Update 3 .rs files that were missing Copyright notices.
Add a section for third_party license notices.
This enables #12687, which will have a race with other code changes.
Type of Change
Which Components or Systems Does This Change Impact?
How Has This Been Tested?
Local testing.
Key Areas to Review
Checklist