microsoft / component-detection

Scans your project to determine what components you use
MIT License
418 stars 86 forks source link

Add Supplier/License info to RustCli Cargo Components. #940

Closed sebasgomez238 closed 8 months ago

sebasgomez238 commented 8 months ago

Adds supplier/license info to Cargo components detected through the RustCli detector. These properties are not made visible in the graph when they are null to avoid null properties on Cargo components detected through the RustCrateDetector.

Approach: Build a dictionary with "{name} {version}" as the key and (license, author) as the value. This allows us to quickly look up the corresponding information when traversing the graph and recording the components.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0923d09) 75.2% compared to head (952a04a) 75.5%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #940 +/- ## ======================================= + Coverage 75.2% 75.5% +0.3% ======================================= Files 234 234 Lines 10052 10064 +12 Branches 990 992 +2 ======================================= + Hits 7560 7601 +41 + Misses 2220 2190 -30 - Partials 272 273 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cobya commented 8 months ago

The rust smoke test failure is unrelated to your changes, so I'll go ahead and merge. Thanks for the contribution!

github-actions[bot] commented 8 months ago

👋 Hi! It looks like you modified some files in the Detectors folder. You may need to bump the detector versions if any of the following scenarios apply:

If none of the above scenarios apply, feel free to ignore this comment 🙂