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

Document `declare_clippy_lint` macro #3758

Merged
merged 3 commits into from Feb 12, 2019

Conversation

Projects
None yet
3 participants
@phansch
Copy link
Collaborator

phansch commented Feb 11, 2019

Split up from my work on updating CONTRIBUTING.md, which is slowly
making progress.

cc #2666

phansch added some commits Feb 11, 2019

Document `declare_clippy_lint` macro
Split up from my work on updating CONTRIBUTING.md, which is slowly
making progress.

cc #2666

@phansch phansch force-pushed the phansch:document_macro branch from 79c167a to 7d39832 Feb 11, 2019

///
/// Pre-expansion lints run before any macro expansion has happened.
///
/// Note that due to the architechture of the compiler, currently `cfg_attr` attributes will still

This comment has been minimized.

@flip1995

flip1995 Feb 11, 2019

Collaborator

That's only true for crate level cfg_attrs.

@flip1995

This comment has been minimized.

Copy link
Collaborator

flip1995 commented Feb 11, 2019

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Feb 11, 2019

📌 Commit a14247b has been approved by flip1995

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Feb 11, 2019

⌛️ Testing commit a14247b with merge 2066368...

bors added a commit that referenced this pull request Feb 11, 2019

Auto merge of #3758 - phansch:document_macro, r=flip1995
Document `declare_clippy_lint` macro

Split up from my work on updating CONTRIBUTING.md, which is slowly
making progress.

cc #2666
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Feb 12, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 2066368 to master...

@bors bors merged commit a14247b into rust-lang:master Feb 12, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.