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

Conversation

@koushiro
Copy link
Contributor

@koushiro koushiro commented Mar 29, 2022

Because TryInto/TryFrom are part of the prelude by default from edition 2021

polkadot companion: paritytech/polkadot#5218

Because `TryInto`/`TryFrom` are in prelude by default from edition 2021

Signed-off-by: koushiro <[email protected]>
@koushiro koushiro changed the title Remove useless imports Remove useless TryInto/TryFrom imports Apr 2, 2022
@ggwpez ggwpez added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Apr 2, 2022
@bkchr bkchr merged commit 01a3ad6 into paritytech:master Apr 3, 2022
@koushiro koushiro deleted the remove-try-from-into branch April 4, 2022 00:01
@koushiro koushiro restored the remove-try-from-into branch April 4, 2022 00:02
@koushiro koushiro deleted the remove-try-from-into branch April 4, 2022 09:14
@athei athei mentioned this pull request Apr 4, 2022
@notlesh
Copy link
Contributor

notlesh commented Apr 27, 2022

PSA: if anyone is running into failed to resolve: use of undeclared type TryInto errors after this PR, it may be that your Cargo.toml specifies an old edition, as discussed here:

https://substrate.stackexchange.com/questions/2193/cannot-find-tryinto-compilation-error-after-it-was-pruned-from-prelude

DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
Because `TryInto`/`TryFrom` are in prelude by default from edition 2021

Signed-off-by: koushiro <[email protected]>
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
Because `TryInto`/`TryFrom` are in prelude by default from edition 2021

Signed-off-by: koushiro <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants