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

0.16: Update CI configuration so that CI jobs pass #1552

Merged
merged 7 commits into from
Nov 8, 2022
Merged

Conversation

briansmith
Copy link
Owner

clippy still fails but I'm thinking to just let it continue to fail on this branch since I don't want to risk accidentally changing the MSRV of 0.16.

@briansmith briansmith self-assigned this Nov 8, 2022
@briansmith briansmith mentioned this pull request Nov 8, 2022
@codecov
Copy link

codecov bot commented Nov 8, 2022

Codecov Report

Merging #1552 (829696d) into b/0.16 (9cc0d45) will decrease coverage by 2.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           b/0.16    #1552      +/-   ##
==========================================
- Coverage   91.79%   89.17%   -2.62%     
==========================================
  Files         106      107       +1     
  Lines       14342    15277     +935     
  Branches        0      185     +185     
==========================================
+ Hits        13165    13624     +459     
- Misses       1177     1618     +441     
- Partials        0       35      +35     
Impacted Files Coverage Δ
src/lib.rs 16.79% <0.00%> (-80.64%) ⬇️
crypto/constant_time_test.c 58.82% <0.00%> (-21.18%) ⬇️
crypto/cpu-intel.c 64.78% <0.00%> (-19.92%) ⬇️
src/aead/chacha20_poly1305_openssh.rs 88.76% <0.00%> (-11.24%) ⬇️
crypto/fipsmodule/bn/montgomery.c 86.36% <0.00%> (-10.07%) ⬇️
crypto/fipsmodule/ec/ecp_nistz.h 90.90% <0.00%> (-9.10%) ⬇️
crypto/internal.h 87.27% <0.00%> (-7.65%) ⬇️
src/io/positive.rs 94.11% <0.00%> (-5.89%) ⬇️
src/endian.rs 85.71% <0.00%> (-5.20%) ⬇️
crypto/fipsmodule/ec/ecp_nistz.c 95.23% <0.00%> (-4.77%) ⬇️
... and 68 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

`git cherry-pick 32e007f`.

Use Android NDK 25 since that's GitHub Actions is moving to. SDK level
is the minimum supported, and there are no longer target-specific
allases for `llvm-ar`; update cargo.sh accordingly.

Use the environment variables recommended by the Android documentation.

Work around compatibility issue with Rust and Android NDKs.

Remove out-of-date documentation aboud building for Android.
@briansmith briansmith changed the base branch from main to b/0.16 November 8, 2022 19:55
@briansmith briansmith merged commit c58b615 into b/0.16 Nov 8, 2022
@briansmith briansmith deleted the b/0.16-cleanup branch November 8, 2022 22:36
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