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

macros: allow attribute invocations at the crate root #36618

Merged
merged 2 commits into from
Sep 22, 2016

Conversation

jseyfried
Copy link
Contributor

Fixes #36617.
r? @nrc

@jseyfried
Copy link
Contributor Author

I unintentionally disallowed attribute macro invocations at the crate root in #36214.

// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![derive(Copy)] //~ ERROR `derive` may only be applied to structs and enums
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be ``derive may only be applied to structs, enums and unions

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@nrc
Copy link
Member

nrc commented Sep 21, 2016

r=me with test fixed

@jseyfried jseyfried force-pushed the crate_root_attr_invoc branch from 2adc803 to f4fa62f Compare September 21, 2016 21:39
@jseyfried
Copy link
Contributor Author

@bors r=nrc

@bors
Copy link
Contributor

bors commented Sep 21, 2016

📌 Commit f4fa62f has been approved by nrc

@bors
Copy link
Contributor

bors commented Sep 22, 2016

⌛ Testing commit f4fa62f with merge a09cb57...

bors added a commit that referenced this pull request Sep 22, 2016
macros: allow attribute invocations at the crate root

Fixes #36617.
r? @nrc
@bors bors merged commit f4fa62f into rust-lang:master Sep 22, 2016
@jseyfried jseyfried deleted the crate_root_attr_invoc branch September 22, 2016 20:10
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.

3 participants