Skip to content

feat(ci): deny cargo doc warnings - #2192

Merged
LesnyRumcajs merged 2 commits into
masterfrom
deny-cargo-doc-warnings
Aug 4, 2025
Merged

feat(ci): deny cargo doc warnings#2192
LesnyRumcajs merged 2 commits into
masterfrom
deny-cargo-doc-warnings

Conversation

@LesnyRumcajs

@LesnyRumcajs LesnyRumcajs commented Aug 1, 2025

Copy link
Copy Markdown
Contributor
  • added cargo doc checks to the CI with fail on warnings,
  • fixed warnings.

Closes #2191

@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Aug 1, 2025
@codecov-commenter

codecov-commenter commented Aug 1, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.50%. Comparing base (9273717) to head (2b3bc2e).
⚠️ Report is 37 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2192   +/-   ##
=======================================
  Coverage   77.50%   77.50%           
=======================================
  Files         147      147           
  Lines       15743    15743           
=======================================
  Hits        12201    12201           
  Misses       3542     3542           
Files with missing lines Coverage Δ
fvm/src/gas/timer.rs 87.03% <ø> (ø)
fvm/src/syscalls/error.rs 52.94% <ø> (ø)
ipld/amt/src/lib.rs 94.11% <ø> (ø)
ipld/bitfield/src/rleplus/mod.rs 90.60% <ø> (ø)
sdk/src/message.rs 0.00% <ø> (ø)
sdk/src/sys/mod.rs 0.00% <ø> (ø)
shared/src/sys/out.rs 0.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LesnyRumcajs
LesnyRumcajs force-pushed the deny-cargo-doc-warnings branch from e0f7aee to 4e5be47 Compare August 1, 2025 11:54
@LesnyRumcajs
LesnyRumcajs marked this pull request as ready for review August 1, 2025 12:18
@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Aug 4, 2025
@LesnyRumcajs
LesnyRumcajs merged commit 93299d5 into master Aug 4, 2025
17 checks passed
@LesnyRumcajs
LesnyRumcajs deleted the deny-cargo-doc-warnings branch August 4, 2025 07:40
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

docs: broken/invalid links

3 participants