Skip to content
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

CI: Address clippy lints and upgrade tools so CI succeeds #2149

Merged
merged 7 commits into from
Sep 28, 2024

Conversation

briansmith
Copy link
Owner

@briansmith briansmith commented Sep 27, 2024

See each individual commit message for details.

@briansmith briansmith self-assigned this Sep 27, 2024
Copy link

codecov bot commented Sep 27, 2024

Codecov Report

Attention: Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 96.97%. Comparing base (7c0024a) to head (56d934f).
Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/polyfill/slice.rs 0.00% 0 Missing and 2 partials ⚠️
src/arithmetic/constant.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2149      +/-   ##
==========================================
- Coverage   97.12%   96.97%   -0.16%     
==========================================
  Files         151      151              
  Lines       20101    19597     -504     
  Branches      447      468      +21     
==========================================
- Hits        19524    19004     -520     
- Misses        482      487       +5     
- Partials       95      106      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Address a new complaint from the newest Clippy.

Remove the relevant outdated/misleading advice in STYLE.md.
0.31.0 doesn't work any more.

Also fix the cache key used in the job.
The older version we have been using is failing to build in CI.
The older version we have been using is failing to build in CI.
@briansmith briansmith force-pushed the b/clippy-1.81 branch 5 times, most recently from 0bac8ee to 0ff704d Compare September 27, 2024 23:34
The version of the Android NDK used in GitHub Actions,
/usr/local/lib/android/sdk/ndk/27.1.12297006, doesn't support
SDK version 19 (or 20) any more.
@briansmith briansmith merged commit 25cc803 into main Sep 28, 2024
156 of 158 checks passed
@briansmith briansmith deleted the b/clippy-1.81 branch September 28, 2024 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant