Skip to content

Drop the From derive macro from the v1 prelude - #150203

Merged
bors merged 1 commit into
rust-lang:mainfrom
jtgeibel:drop-From-derive-macro-from-prelude
Dec 22, 2025
Merged

Drop the From derive macro from the v1 prelude#150203
bors merged 1 commit into
rust-lang:mainfrom
jtgeibel:drop-From-derive-macro-from-prelude

Conversation

@jtgeibel

@jtgeibeljtgeibel commented Dec 20, 2025

Copy link
Copy Markdown
Member

This was accidentally added to the prelude in 3f4dc1e.

Fixes: #150165

r? @jdonszelmann

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Dec 20, 2025
@rustbot

This comment has been minimized.

This was accidentally added to the prelude in
3f4dc1e.
@Kivooeo

Kivooeo commented Dec 20, 2025

Copy link
Copy Markdown
Member

It was added in #144922 as a part of implementation this feature

So as far as I understand it shouldn't be removed

r? Kobzol

@rustbotrustbot assigned Kobzol and unassigned jdonszelmannDec 20, 2025
@rustbot

Copy link
Copy Markdown
Collaborator

Kobzol is not on the review rotation at the moment.
They may take a while to respond.

@jdonszelmann

Copy link
Copy Markdown
Contributor

No I think it got removed from the prelude again because of the shadowing bug demonstrated right there. My addition was just a rebasing bug. @Kobzol r=me if that's true

@Kobzol

Copy link
Copy Markdown
Member

It should actually be removed, because it is breaking users 😅 It was added in #144922, but got later removed to fix this. I was wondering why the problem reappeared, #146348 explains it.

@jdonszelmann

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Dec 20, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit 93fbf3b has been approved by jdonszelmann

It is now in the queue for this repository.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 20, 2025
Kivooeo added a commit to Kivooeo/rust that referenced this pull request Dec 20, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? `@jdonszelmann`
@KivooeoKivooeo mentioned this pull request Dec 20, 2025
bors added a commit that referenced this pull request Dec 20, 2025
Rollup of 7 pull requests
Successful merges:
- #147552 ([Debugger Visualizers] Optimize lookup behavior)
- #149437 (Fix trailing newline in JUnit formatter)
- #149812 (Add const default for OnceCell and OnceLock)
- #150035 (fix docustring on fetch_or)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
Kivooeo added a commit to Kivooeo/rust that referenced this pull request Dec 20, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? `@jdonszelmann`
@KivooeoKivooeo mentioned this pull request Dec 20, 2025
bors added a commit that referenced this pull request Dec 21, 2025
Rollup of 6 pull requests
Successful merges:
- #149437 (Fix trailing newline in JUnit formatter)
- #149812 (Add const default for OnceCell and OnceLock)
- #150035 (fix docustring on fetch_or)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
Zalathar added a commit to Zalathar/rust that referenced this pull request Dec 21, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? ``@jdonszelmann``
@ZalatharZalathar mentioned this pull request Dec 21, 2025
bors added a commit that referenced this pull request Dec 21, 2025
Rollup of 13 pull requests
Successful merges:
- #146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- #149437 (Fix trailing newline in JUnit formatter)
- #149658 (tests/assembly-llvm/some-non-zero-from-atomic-optimization.rs: New test)
- #149812 (Add const default for OnceCell and OnceLock)
- #149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- #150009 (Enable llvm-libunwind by default for Hexagon targets)
- #150035 (fix docustring on fetch_or)
- #150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
- #150208 (Update books)
r? `@ghost`
`@rustbot` modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Dec 21, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? ```@jdonszelmann```
bors added a commit that referenced this pull request Dec 21, 2025
Rollup of 11 pull requests
Successful merges:
- #146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- #149658 (tests/assembly-llvm/some-non-zero-from-atomic-optimization.rs: New test)
- #149812 (Add const default for OnceCell and OnceLock)
- #149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- #150009 (Enable llvm-libunwind by default for Hexagon targets)
- #150035 (fix docustring on fetch_or)
- #150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Dec 21, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? ````@jdonszelmann````
bors added a commit that referenced this pull request Dec 21, 2025
…uwer
Rollup of 13 pull requests
Successful merges:
- #146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- #149812 (Add const default for OnceCell and OnceLock)
- #149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- #150009 (Enable llvm-libunwind by default for Hexagon targets)
- #150035 (fix docustring on fetch_or)
- #150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- #150185 ([rustdoc] Add missing close tags in extern crate reexports)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
- #150208 (Update books)
- #150210 (tests/debuginfo/function-arg-initialization.rs: Stop disabling SingleUseConsts MIR pass)
r? `@ghost`
`@rustbot` modify labels: rollup
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Dec 21, 2025
…om-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: rust-lang#150165
r? ``````@jdonszelmann``````
bors added a commit that referenced this pull request Dec 21, 2025
Rollup of 11 pull requests
Successful merges:
- #146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- #149812 (Add const default for OnceCell and OnceLock)
- #149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- #150009 (Enable llvm-libunwind by default for Hexagon targets)
- #150035 (fix docustring on fetch_or)
- #150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- #150185 ([rustdoc] Add missing close tags in extern crate reexports)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Dec 22, 2025
Rollup of 11 pull requests
Successful merges:
- #146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- #149812 (Add const default for OnceCell and OnceLock)
- #149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- #150009 (Enable llvm-libunwind by default for Hexagon targets)
- #150035 (fix docustring on fetch_or)
- #150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- #150160 (Fix ICE (#149980) for invalid EII in statement position)
- #150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- #150185 ([rustdoc] Add missing close tags in extern crate reexports)
- #150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- #150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit c91cbcd into rust-lang:mainDec 22, 2025
11 checks passed
rust-timer added a commit that referenced this pull request Dec 22, 2025
Rollup merge of #150203 - jtgeibel:drop-From-derive-macro-from-prelude, r=jdonszelmann
Drop the From derive macro from the v1 prelude
This was accidentally added to the prelude in 3f4dc1e.
Fixes: #150165
r? `````@jdonszelmann`````
@rustbotrustbot added this to the 1.94.0 milestone Dec 22, 2025
github-actionsBot pushed a commit to rust-lang/miri that referenced this pull request Dec 23, 2025
Rollup of 11 pull requests
Successful merges:
- rust-lang/rust#146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- rust-lang/rust#149812 (Add const default for OnceCell and OnceLock)
- rust-lang/rust#149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- rust-lang/rust#150009 (Enable llvm-libunwind by default for Hexagon targets)
- rust-lang/rust#150035 (fix docustring on fetch_or)
- rust-lang/rust#150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- rust-lang/rust#150160 (Fix ICE (rust-lang/rust#149980) for invalid EII in statement position)
- rust-lang/rust#150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- rust-lang/rust#150185 ([rustdoc] Add missing close tags in extern crate reexports)
- rust-lang/rust#150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- rust-lang/rust#150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Aug 17, 2026
Rollup of 11 pull requests
Successful merges:
- rust-lang/rust#146377 (Don't strip shebang in expr-ctxt `include!(…)`)
- rust-lang/rust#149812 (Add const default for OnceCell and OnceLock)
- rust-lang/rust#149882 (miri: add -Zbinary-dep-depinfo to dependency builds)
- rust-lang/rust#150009 (Enable llvm-libunwind by default for Hexagon targets)
- rust-lang/rust#150035 (fix docustring on fetch_or)
- rust-lang/rust#150082 (tests/ui/traits/fmt-pointer-trait.rs: Add HRTB fn pointer case)
- rust-lang/rust#150160 (Fix ICE (rust-lang/rust#149980) for invalid EII in statement position)
- rust-lang/rust#150184 (mir_build: Use the same length type for `TestableCase::Slice` and `TestKind::Len`)
- rust-lang/rust#150185 ([rustdoc] Add missing close tags in extern crate reexports)
- rust-lang/rust#150191 (change non-canonical clone impl to {*self}, fix some doc comments)
- rust-lang/rust#150203 (Drop the From derive macro from the v1 prelude)
r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Derive macro glob import flagged as ambiguous with implicit prelude import

6 participants

@jtgeibel@rustbot@Kivooeo@jdonszelmann@Kobzol@bors