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

build.rs: use rustfmt #2150

Closed
Arjentix opened this issue Apr 26, 2022 · 0 comments
Closed

build.rs: use rustfmt #2150

Arjentix opened this issue Apr 26, 2022 · 0 comments
Assignees
Labels
build good first issue Good for newcomers iroha2-dev The re-implementation of a BFT hyperledger in RUST

Comments

@Arjentix
Copy link
Contributor

Right now client/build.rs uses cargo to run cargo fmt. This approach can cause cargo recursion problem (cargo doesn't like to be invoked multiple times from it self) especially when using cargo lints. So we need to use rustfmt instead of cargo fmt.

To replace cargo build inside this script we need to do more work and that's gonna be in another issue

@Arjentix Arjentix added good first issue Good for newcomers iroha2-dev The re-implementation of a BFT hyperledger in RUST build labels Apr 26, 2022
@Arjentix Arjentix self-assigned this May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build good first issue Good for newcomers iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Development

No branches or pull requests

1 participant