Unsupport #[derive(Trait)] sugar for #[derive_Trait] legacy plugin attributes#54271
Unsupport #[derive(Trait)] sugar for #[derive_Trait] legacy plugin attributes#54271bors merged 1 commit intorust-lang:masterfrom
#[derive(Trait)] sugar for #[derive_Trait] legacy plugin attributes#54271Conversation
|
☔ The latest upstream changes (presumably #53461) made this pull request unmergeable. Please resolve the merge conflicts. |
|
cc @SergioBenitez who reported about the feature still being used in Rocker in July. That said, |
|
We've migrated all of Rocket's derives to procedural macros. That being said, that migration is not yet part of an official release, so this would be a breaking change for almost all of Rocket's users. If it's possible to delay this PR until Rocket sees a new release, which should happen in a matter of weeks as there is only a single blocking feature remaining, that would be much appreciated. |
|
@SergioBenitez |
|
@petrochenkov The |
|
@SergioBenitez what's the status now? |
|
@Centril We are no longer using this in Rocket as of the the 0.4 release candidate. The GA of 0.4 has not yet been released; that should happen on the 14th. Ideally this wouldn't be merged until some time after the GA is available to give users a chance to migrate. |
|
Rocket 0.4 is still not GA (is currently on -rc2, which has been published yesterday). |
52d0c28 to
8ab115c
Compare
|
📌 Commit 8ab115c has been approved by |
Unsupport `#[derive(Trait)]` sugar for `#[derive_Trait]` legacy plugin attributes This is a long deprecated unstable feature that doesn't mesh well with regular resolution/expansion. How to fix broken code: - The recommended way is to migrate to stable procedural macros - derives or attributes (https://doc.rust-lang.org/nightly/book/first-edition/procedural-macros.html). - If that's not possible right now for some reason, you can keep code working with a simple mechanical replacement `#[derive(Legacy)]` -> `#[derive_Legacy]`. Closes #29644 r? @ghost
|
☀️ Test successful - status-appveyor, status-travis |
This is a long deprecated unstable feature that doesn't mesh well with regular resolution/expansion.
How to fix broken code:
rocket 0.3.*in particular, the recommended solution is to migrate torocket 0.4.*.#[derive(Legacy)]->#[derive_Legacy].Closes #29644
r? @ghost