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

Add a test for #2526 #3811

Merged
merged 1 commit into from
Feb 24, 2019
Merged

Add a test for #2526 #3811

merged 1 commit into from
Feb 24, 2019

Conversation

mcarton
Copy link
Member

@mcarton mcarton commented Feb 24, 2019

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.

@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Feb 24, 2019

📌 Commit a05f305 has been approved by flip1995

@mcarton
Copy link
Member Author

mcarton commented Feb 24, 2019

I'm an idiot, I was fiddling with the attribute, to check for other crate type and forgot to restore it before committing 😅

@flip1995
Copy link
Member

flip1995 commented Feb 24, 2019

Oh I also completely missed that! Thanks for the fast fix. Now this should be good to go.

@bors r+

@bors

This comment has been minimized.

@bors

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Feb 24, 2019

📌 Commit 3a9426e has been approved by flip1995

@bors
Copy link
Contributor

bors commented Feb 24, 2019

⌛ Testing commit 3a9426e with merge f7fa2a8...

bors added a commit that referenced this pull request Feb 24, 2019
Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.
@bors
Copy link
Contributor

bors commented Feb 24, 2019

💔 Test failed - checks-travis

@mcarton
Copy link
Member Author

mcarton commented Feb 24, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Feb 24, 2019

📌 Commit 8bcd546 has been approved by mcarton

@bors
Copy link
Contributor

bors commented Feb 24, 2019

⌛ Testing commit 8bcd546 with merge e99611b...

bors added a commit that referenced this pull request Feb 24, 2019
Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.
@mcarton
Copy link
Member Author

mcarton commented Feb 24, 2019

The test had an extra blank line…

@bors
Copy link
Contributor

bors commented Feb 24, 2019

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

@bors bors merged commit 8bcd546 into master Feb 24, 2019
@mcarton mcarton deleted the test-for-2526 branch February 24, 2019 18:55
@oli-obk
Copy link
Contributor

oli-obk commented Feb 24, 2019

Yea, you can r+ your own PRs. The convention is to r=actual_reviewer instead of r+

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.

Clippy does not run on proc macro crates
4 participants