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: doc build fix, update actions/checkout #1355

Merged
merged 3 commits into from
Sep 19, 2023
Merged

CI: doc build fix, update actions/checkout #1355

merged 3 commits into from
Sep 19, 2023

Conversation

mzabaluev
Copy link
Contributor

Fix a breakage in the cargo doc check.
Also update to actions/checkout@v4.

@codecov-commenter
Copy link

Codecov Report

Merging #1355 (02b8ee2) into main (ff7a864) will decrease coverage by 0.5%.
The diff coverage is n/a.

❗ Current head 02b8ee2 differs from pull request most recent head 5a094f9. Consider uploading reports for the commit 5a094f9 to get more accurate results

@@           Coverage Diff           @@
##            main   #1355     +/-   ##
=======================================
- Coverage   60.3%   59.8%   -0.5%     
=======================================
  Files        275     274      -1     
  Lines      27037   27262    +225     
=======================================
  Hits       16314   16314             
- Misses     10723   10948    +225     

see 4 files with indirect coverage changes

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

The docs now build on stable, and nightly triggers some bug with a
dependency.
One dependency has docs that make nightly rustdoc fail,
while others require nightly with the "docsrs" feature.
@mzabaluev mzabaluev merged commit 6d4e538 into main Sep 19, 2023
23 checks passed
@mzabaluev mzabaluev deleted the mikhail/ci-fixes branch September 19, 2023 13:49
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.

3 participants