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

account for external macro in MISSING_INLINE_IN_PUBLIC_ITEMS lint #4870

Merged

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Dec 1, 2019

Closes #4861
changelog: Fix FP (external macro) in [missing_inline_in_public_items] lint

@flip1995
Copy link
Member

flip1995 commented Dec 1, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Dec 1, 2019

📌 Commit 5488672 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Dec 1, 2019

⌛ Testing commit 5488672 with merge 5a75502...

bors added a commit that referenced this pull request Dec 1, 2019
…ms, r=flip1995

account for external macro in MISSING_INLINE_IN_PUBLIC_ITEMS lint

Closes #4861
changelog: Fix FP (external macro) in [`missing_inline_in_public_items`] lint
@bors
Copy link
Contributor

bors commented Dec 1, 2019

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

@bors bors merged commit 5488672 into rust-lang:master Dec 1, 2019
@tesuji tesuji deleted the check-macro-missing_inline_in_public_items branch December 1, 2019 18:41
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.

missing_inline_in_public_items warns about #[derive]-generated code
3 participants