Skip to content

Revert "drop derive helpers during ast lowering" #157150

Open
JonathanBrouwer wants to merge 2 commits into
rust-lang:mainfrom
JonathanBrouwer:fix-attrs
Open

Revert "drop derive helpers during ast lowering" #157150
JonathanBrouwer wants to merge 2 commits into
rust-lang:mainfrom
JonathanBrouwer:fix-attrs

Conversation

@JonathanBrouwer
Copy link
Copy Markdown
Contributor

Fixes #157107
By undoing commit 29e9273 from #153540

cc @scrabsha @jdonszelmann

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 30, 2026

These commits modify tests/rustdoc-json.
rustdoc-json is a public (but unstable) interface.

Please ensure that if you've changed the output:

  • It's intentional.
  • The FORMAT_VERSION in src/librustdoc-json-types is bumped if necessary.

cc @aDotInTheVoid, @obi1kenobi

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) A-rustdoc-json Area: Rustdoc JSON backend S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels May 30, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 30, 2026

r? @oli-obk

rustbot has assigned @oli-obk.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 73 candidates
  • Random selection from 16 candidates

@JonathanBrouwer JonathanBrouwer marked this pull request as draft May 30, 2026 10:35
@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 30, 2026
@jdonszelmann
Copy link
Copy Markdown
Contributor

I hate this but it's the right thing to do for now. We can fix it later once we work out how

@bors r+ rollup

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 30, 2026

📋 Only open, non-draft PRs can be approved.

@jdonszelmann
Copy link
Copy Markdown
Contributor

Oh, r=me in that case

@jdonszelmann jdonszelmann assigned jdonszelmann and unassigned oli-obk May 30, 2026
@JonathanBrouwer JonathanBrouwer marked this pull request as ready for review May 30, 2026 10:56
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels May 30, 2026
@JonathanBrouwer
Copy link
Copy Markdown
Contributor Author

Let's revert first and then see if there's a better solution later
@bors r=jdonszelmann

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 30, 2026

📌 Commit 5be8451 has been approved by jdonszelmann

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 May 30, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…zelmann

Revert "drop derive helpers during ast lowering"

Fixes rust-lang#157107
By undoing commit rust-lang@29e9273 from rust-lang#153540

cc @scrabsha @jdonszelmann
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…zelmann

Revert "drop derive helpers during ast lowering"

Fixes rust-lang#157107
By undoing commit rust-lang@29e9273 from rust-lang#153540

cc @scrabsha @jdonszelmann
rust-bors Bot pushed a commit that referenced this pull request May 30, 2026
…uwer

Rollup of 8 pull requests

Successful merges:

 - #156863 (Make hint::cold_path #[cold] so that it works even if the MIR inliner can't inline it)
 - #156875 (Correct and document semantics of `yield` terminator)
 - #157115 ([rustdoc] Fix foreign items macro expansion)
 - #157150 (Revert "drop derive helpers during ast lowering" )
 - #156887 (Rename `-Zdebuginfo-for-profiling` switch)
 - #157039 (rustdoc: correctly propagate cfgs for glob reexports)
 - #157125 (Rewrite the `#[repr]` attribute parser)
 - #157154 (Revert workaround used to select the gcc codegen in the coretests CI)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) A-rustdoc-json Area: Rustdoc JSON backend S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

derive helpers not in attrs field in rustdoc json

4 participants