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 macro expansion in toplevel_ref_arg lint #4570

Merged
merged 3 commits into from
Sep 26, 2019

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Sep 25, 2019

changelog: Fix macro expansion in toplevel_ref_arg lint

@phansch phansch added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 25, 2019
@tesuji tesuji force-pushed the call_site_toplevel_ref_arg branch from 2874169 to 11fadcf Compare September 26, 2019 01:14
@tesuji tesuji force-pushed the call_site_toplevel_ref_arg branch from 11fadcf to 5639639 Compare September 26, 2019 01:54
@phansch
Copy link
Member

phansch commented Sep 26, 2019

@bors r+ thanks!

@bors
Copy link
Contributor

bors commented Sep 26, 2019

📌 Commit 5639639 has been approved by phansch

@bors
Copy link
Contributor

bors commented Sep 26, 2019

⌛ Testing commit 5639639 with merge bc1b04b...

bors added a commit that referenced this pull request Sep 26, 2019
Fix macro expansion in toplevel_ref_arg lint

changelog: Fix macro expansion in toplevel_ref_arg lint
@bors
Copy link
Contributor

bors commented Sep 26, 2019

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

@bors bors merged commit 5639639 into rust-lang:master Sep 26, 2019
@tesuji tesuji deleted the call_site_toplevel_ref_arg branch September 26, 2019 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants