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

fix nightly warning legacy_derive_helpers #7648

Merged
merged 1 commit into from
Feb 12, 2021

Conversation

peddermaster2
Copy link

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
Copy link
Member

lnicola commented Feb 12, 2021

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

@jonas-schievink
Copy link
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 <[email protected]>
Co-authored-by: Peter Wischer <[email protected]>
@bors
Copy link
Contributor

bors bot commented Feb 12, 2021

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 <[email protected]>
@bors
Copy link
Contributor

bors bot commented Feb 12, 2021

Build failed:

@lnicola
Copy link
Member

lnicola commented Feb 12, 2021

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

@peddermaster2
Copy link
Author

Done.

@lnicola
Copy link
Member

lnicola commented Feb 12, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Feb 12, 2021

Canceled.

@lnicola
Copy link
Member

lnicola commented Feb 12, 2021

bors r+

There was no need to rebase.

@peddermaster2
Copy link
Author

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

@bors
Copy link
Contributor

bors bot commented Feb 12, 2021

@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