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

Add support for rust via cargo #404

Merged
merged 2 commits into from
Sep 19, 2021
Merged

Add support for rust via cargo #404

merged 2 commits into from
Sep 19, 2021

Conversation

jonabc
Copy link
Contributor

@jonabc jonabc commented Sep 19, 2021

This change adds support for rust using the cargo package manager. Generally this is one of the simplest source implementations in a long time 🎉 . A HUGE HUGE thank you to the rust and cargo developers for providing all the metadata this tool needs from a single cargo metadata CLI call, and returning data in an easy to consume format.

There is an extensibility option available with cargo.metadata_options configuration. This key can be set to a string or an array of strings, and will be appended to the end of the default CLI call cargo metadata --format-version=1.

The source is pretty straightforward - it pulls the ids from resolved.nodes output and matches them up to package data given in packages, excluding any packages who's ids are present in workspace_members as 1st party packages that don't need reporting on. All of the metadata that licensed needs is available in the packages object ❤️ .

@jonabc jonabc merged commit 9f75185 into master Sep 19, 2021
@jonabc jonabc deleted the rusty branch September 19, 2021 03:08
jonabc added a commit that referenced this pull request Sep 19, 2021
2021-09-18

### Added

- New cargo source enumerates rust dependencies (#404)

### Changed

- Removed non-functional files from gem builds (#405)
@jonabc jonabc mentioned this pull request Sep 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant