Skip to content

feat: include build and dev dependencies - #392

Open
wichert wants to merge 1 commit into
psastras:mainfrom
wichert:devel-dependencies
Open

feat: include build and dev dependencies#392
wichert wants to merge 1 commit into
psastras:mainfrom
wichert:devel-dependencies

Conversation

@wichert

@wichert wichert commented Feb 3, 2026

Copy link
Copy Markdown

This extends the SBOM to include build and dev dependencies.

This extends the SBOM to include build and dev dependencies.
@codecov-commenter

codecov-commenter commented Feb 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 6.34%. Comparing base (b616135) to head (bca70ec).
⚠️ Report is 375 commits behind head on main.

Files with missing lines Patch % Lines
cargo-sbom/src/util/spdx/mod.rs 0.00% 31 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             main    #392      +/-   ##
=========================================
- Coverage   12.94%   6.34%   -6.60%     
=========================================
  Files           7       5       -2     
  Lines         595     946     +351     
  Branches      595     946     +351     
=========================================
- Hits           77      60      -17     
- Misses        499     877     +378     
+ Partials       19       9      -10     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@psastras psastras left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, minor issue.

fn pi() -> f64 {
3.1415926
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mistake? If so can you remove?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, this was deliberate to add an example of a function with a test that uses a dev dependency (approx).

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