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

remove unnessisary use statements due to 2021 core prelude #4183

Merged
merged 7 commits into from
Apr 28, 2024

Conversation

gilescope
Copy link
Contributor

@gilescope gilescope commented Apr 18, 2024

Some traits are already included in the 2021 prelude and so shouldn't be needed to use explicitly:

use convert::TryFrom, convert::TryInto, and iter::FromIterator are removed.

( https://doc.rust-lang.org/core/prelude/rust_2021/ )

No breaking changes or change of functionality, so I think no PR doc is needed in this case.

(Motivation: Removes some references to sp-std)

@gilescope gilescope marked this pull request as ready for review April 19, 2024 08:02
@gilescope gilescope requested review from andresilva and a team as code owners April 19, 2024 08:02
@paritytech-review-bot paritytech-review-bot bot requested a review from a team April 19, 2024 08:03
Copy link
Member

@ggwpez ggwpez left a comment

Choose a reason for hiding this comment

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

sp-core-fuzzer is on edition 2015. Could you please also fix that while youre at it?

@gilescope gilescope requested a review from athei as a code owner April 21, 2024 08:39
@gilescope
Copy link
Contributor Author

Good spot. There were a few editions set to 2021 but not pointing to the workspace so I've repointed them also.

@bkchr bkchr added the R0-silent Changes should not be mentioned in any release notes label Apr 21, 2024
@seadanda seadanda added the T1-FRAME This PR/Issue is related to core FRAME, the framework. label Apr 24, 2024
@seadanda seadanda enabled auto-merge April 24, 2024 07:30
@gilescope
Copy link
Contributor Author

@bkontur I need a thumbs-up from the bridges team if you're ok with the changes?

@seadanda seadanda added this pull request to the merge queue Apr 28, 2024
Merged via the queue into paritytech:master with commit 954150f Apr 28, 2024
138 of 140 checks passed
Morganamilo pushed a commit that referenced this pull request May 2, 2024
Some traits are already included in the 2021 prelude and so shouldn't be
needed to use explicitly:

use `convert::TryFrom`, `convert::TryInto`, and `iter::FromIterator` are
removed.

( https://doc.rust-lang.org/core/prelude/rust_2021/ )

No breaking changes or change of functionality, so I think no PR doc is
needed in this case.

(Motivation: Removes some references to `sp-std`)
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
…h#4183)

Some traits are already included in the 2021 prelude and so shouldn't be
needed to use explicitly:

use `convert::TryFrom`, `convert::TryInto`, and `iter::FromIterator` are
removed.

( https://doc.rust-lang.org/core/prelude/rust_2021/ )

No breaking changes or change of functionality, so I think no PR doc is
needed in this case.

(Motivation: Removes some references to `sp-std`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T1-FRAME This PR/Issue is related to core FRAME, the framework.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants