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

Omit proc macros from must_use_candidate #4696

Merged
merged 1 commit into from
Oct 18, 2019
Merged

Conversation

llogiq
Copy link
Contributor

@llogiq llogiq commented Oct 18, 2019

This fixes #4684.

changelog: none

@llogiq
Copy link
Contributor Author

llogiq commented Oct 18, 2019

CI fail seems to be unrelated.

@llogiq llogiq requested a review from flip1995 October 18, 2019 15:10
@phansch
Copy link
Member

phansch commented Oct 18, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Oct 18, 2019

📌 Commit d723b35 has been approved by phansch

@bors
Copy link
Contributor

bors commented Oct 18, 2019

⌛ Testing commit d723b35 with merge 850ad52...

bors added a commit that referenced this pull request Oct 18, 2019
Omit proc macros from `must_use_candidate`

This fixes #4684.

changelog: none
@bors
Copy link
Contributor

bors commented Oct 18, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing 850ad52 to master...

@bors bors merged commit d723b35 into master Oct 18, 2019
@llogiq llogiq deleted the need-not-use-proc-macro branch October 18, 2019 20:39
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.

must_use_candidate should not warn proc macros
3 participants