Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

add some common rust tools #1450

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add some common rust tools #1450

wants to merge 2 commits into from

Conversation

lizelive
Copy link
Contributor

No description provided.

@bamurtaugh
Copy link
Member

Thanks for your PR @lizelive! Do you have any additional info on how this update helps or changes the Rust dev experience?

@lizelive
Copy link
Contributor Author

lizelive commented May 26, 2022

yeah most of these are included in github actor default image:

  • bindgen - makes binds
  • cbindgen - makes binds
  • cargo-outdated - tells you if deps are old
  • cargo-audit - run security audit

extra:

  • cargo-edit : A utility for managing cargo dependencies from the command line.
  • cargo-diet: Make your crate lean by computing size-optimal include directives for Cargo manifests

cargo-chef is not meant to be run locally
Its primary use-case is to speed up container builds by running BEFORE the actual source code is copied over. Don't run it on existing codebases to avoid having files being overwritten.
@lizelive
Copy link
Contributor Author

lizelive commented May 26, 2022

after further testing i made changes to the pr:

added:

  • cargo-about: generates a 'about' legal file

removed:

  • cargo-chef: allows building deps (like 10x performance boost when building rust stuff in containers), typically not installed in dev container. i proposed because i use my devcontainer as build container, but not sure if that's typical use case.

@lizelive
Copy link
Contributor Author

github action installed software note: you already install clippy and rustfmt

@bamurtaugh
Copy link
Member

Thanks for the insight - what do you mean by github actor default image?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants