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

[feature] #2048: Add toolchain file #2049

Merged
merged 4 commits into from
Apr 4, 2022

Conversation

Arjentix
Copy link
Contributor

@Arjentix Arjentix commented Apr 1, 2022

Description of the Change

Added toolchain file setting up rustc to version 1.57

Issue

Closes #2048

Benefits

Now project can be build without additional steps like rustup override

Possible Drawbacks

Not a drawback, but at first I thought that will be a way to set different toolchains for different tools.
Unfortunatly we still have to use cargo +<toolchain> ... for clippy and fmt.

This situation looks a bit like clippy problem with multiple sub-crates, so that we have to use cargo lints

@github-actions github-actions bot added the iroha2-dev The re-implementation of a BFT hyperledger in RUST label Apr 1, 2022
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
mversic
mversic previously approved these changes Apr 1, 2022
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
@appetrosyan appetrosyan merged commit af5c8c2 into hyperledger:iroha2-dev Apr 4, 2022
mversic pushed a commit to mversic/iroha that referenced this pull request May 2, 2022
appetrosyan pushed a commit to appetrosyan/iroha that referenced this pull request May 12, 2022
appetrosyan pushed a commit to appetrosyan/iroha that referenced this pull request May 12, 2022
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
appetrosyan pushed a commit to appetrosyan/iroha that referenced this pull request May 12, 2022
Signed-off-by: Daniil Polyakov <arjentix@gmail.com>
mversic pushed a commit to mversic/iroha that referenced this pull request May 13, 2022
mversic pushed a commit to mversic/iroha that referenced this pull request May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants