Skip to content

fix nightly warning legacy_derive_helpers - #7648

Merged
bors[bot] merged 1 commit into
rust-lang:masterfrom
peddermaster2:fix-procmacro-ordering
Feb 12, 2021
Merged

bors[bot] merged 1 commit into
rust-lang:masterfrom
peddermaster2:fix-procmacro-ordering

Conversation

@peddermaster2

Copy link
Copy Markdown

With a recent nightly (e.g. 2021-02-10) a warning comes up. This PR reorders the attributes to fix the warning.

See rust-lang/rust#79202

@lnicola

lnicola commented Feb 12, 2021

Copy link
Copy Markdown
Member

Thanks! It looks good, but we have some CI issues that aren't related to your change.

@jonas-schievink

Copy link
Copy Markdown
Contributor

bors r+

bors Bot added a commit that referenced this pull request Feb 12, 2021
7644: Primitive completion r=jonas-schievink a=jonas-schievink

Fixes #7642

7648: fix nightly warning `legacy_derive_helpers` r=jonas-schievink a=peddermaster2

With a recent nightly (e.g. 2021-02-10) a warning comes up. This PR reorders the attributes to fix the warning.

See rust-lang/rust#79202

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Co-authored-by: Peter Wischer <peter.wischer@lwl.org>
@bors

bors Bot commented Feb 12, 2021

Copy link
Copy Markdown
Contributor

Build failed (retrying...):

bors Bot added a commit that referenced this pull request Feb 12, 2021
7648: fix nightly warning `legacy_derive_helpers` r=jonas-schievink a=peddermaster2

With a recent nightly (e.g. 2021-02-10) a warning comes up. This PR reorders the attributes to fix the warning.

See rust-lang/rust#79202

Co-authored-by: Peter Wischer <peter.wischer@lwl.org>
@bors

bors Bot commented Feb 12, 2021

Copy link
Copy Markdown
Contributor

Build failed:

@lnicola

lnicola commented Feb 12, 2021 •

Copy link
Copy Markdown
Member

Ugh. Can you update the hash in lsp-extensions.md to 8f1ae8530f69e3a3?

@peddermaster2

Copy link
Copy Markdown
Author

Done.

@lnicola

lnicola commented Feb 12, 2021

Copy link
Copy Markdown
Member

bors r+

@bors

bors Bot commented Feb 12, 2021

Copy link
Copy Markdown
Contributor

Canceled.

@lnicola

lnicola commented Feb 12, 2021

Copy link
Copy Markdown
Member

bors r+

There was no need to rebase.

@peddermaster2

Copy link
Copy Markdown
Author

It was getting the same errors with the formatting as before.

@bors

bors Bot commented Feb 12, 2021

Copy link
Copy Markdown
Contributor

@bors
bors Bot merged commit a044ac9 into rust-lang:master Feb 12, 2021
@peddermaster2
peddermaster2 deleted the fix-procmacro-ordering branch February 12, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants