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

Library ification (Phase #1) #77

Merged
merged 4 commits into from
Dec 9, 2021
Merged

Library ification (Phase #1) #77

merged 4 commits into from
Dec 9, 2021

Conversation

amy-keibler
Copy link
Collaborator

This implements the first phase of #68 by moving the CLI application to cargo-cyclonedx/

Restructuring

  • Move all relevant files to cargo-cyclonedx/
  • Update the Cargo.toml to point to the new crate name
  • Create a top-level Cargo.toml that describes the workspace
  • Create a new README.md that will be specific to the CLI application

GitHub Actions changes

  • Make the action cargo-cyclonedx specific
    • Bumped directly rather than bumping the version of everything in the workspace
    • Tagged & committed as cargo-cyclonedx-<version>
    • This will prepare us for creating a second action to deploy cyclonedx-bom that will also open a PR that increments the cargo-cyclonedx dependency
  • Use jq to perform JSON manipulation to get the version

Additional Changes

  • Remove the unused CLI arguments, since this is a new crate that has no users to worry about backwards compatibility

This is a part of the effort mentioned in
#68

Signed-off-by: Amy Keibler <amelia.keibler@gmail.com>
Use jq to parse the JSON from cargo metadata and get the version that way
Update the publish command to publish cargo-cyclonedx
Update the tag and commit to be cargo-cyclonedx specific

This is in preparation of creating a separate workflow that will publish the
library and open a PR to bump the cargo-cyclonedx dependency

Signed-off-by: Amy Keibler <amelia.keibler@gmail.com>
Since this is a new crate, we can make this change without breaking anyone's
workflow

Signed-off-by: Amy Keibler <amelia.keibler@gmail.com>
Signed-off-by: Amy Keibler <amelia.keibler@gmail.com>
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.

None yet

1 participant