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 AsRef and AsMut for AccountId20 #1139

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

boundless-forest
Copy link
Collaborator

@sorpaas sorpaas merged commit 22aaafe into polkadot-evm:master Aug 1, 2023
@boundless-forest boundless-forest deleted the bear-account-asref branch August 1, 2023 13:17
components = ["rustfmt", "clippy"]
targets = ["wasm32-unknown-unknown"]
profile = "minimal"
channel = "nightly-2023-05-22"
Copy link
Collaborator

@koushiro koushiro Aug 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@boundless-forest why change the rust-toolchain config in this PR?

Copy link
Collaborator

@koushiro koushiro Aug 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use the same rust toolchain config of polkadot releases (https://github.com/paritytech/polkadot/releases/tag/v1.0.0)

channel="nightly-2023-05-22" is not 8b4b20836

rustup show

nightly-2023-05-23-aarch64-apple-darwin
rustc 1.71.0-nightly (8b4b20836 2023-05-22)

Copy link
Collaborator Author

@boundless-forest boundless-forest Aug 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nightly-2023-05-22 is not 8b4b20836

Why? rustc 1.71.0-nightly (8b4b20836 2023-05-22) I understand the 8b4b20836 is the nightly version id of the 2023-05-22.

Is this output misleading?

Copy link
Collaborator

@koushiro koushiro Aug 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you could try to run the command rustup show in the frontier workspace and you didn't change the toolchain of CI at the same time.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. I see your point. How about updating this later in next PR?

boundless-forest added a commit to boundless-forest/frontier that referenced this pull request Aug 3, 2023
* Update toolchain toml config

* Add helper functions
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.

3 participants