Skip to content

a few minor cleanups #4946

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

Merged
merged 1 commit into from
Dec 23, 2019
Merged

a few minor cleanups #4946

merged 1 commit into from
Dec 23, 2019

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Dec 23, 2019

changelog: none

@flip1995
Copy link
Member

r=me once travis passes

@bors delegate+

@bors
Copy link
Contributor

bors commented Dec 23, 2019

✌️ @lzutao can now approve this pull request

@tesuji tesuji force-pushed the take branch 3 times, most recently from 8f54ebc to 3e84ffe Compare December 23, 2019 19:46
@tesuji
Copy link
Contributor Author

tesuji commented Dec 23, 2019

@bors r=flip1995

@bors
Copy link
Contributor

bors commented Dec 23, 2019

📌 Commit 3e84ffe has been approved by flip1995

@bors
Copy link
Contributor

bors commented Dec 23, 2019

⌛ Testing commit 3e84ffe with merge 61d5707...

bors added a commit that referenced this pull request Dec 23, 2019
a few minor cleanups

changelog: none
@tesuji
Copy link
Contributor Author

tesuji commented Dec 23, 2019

@bors r-

@bors
Copy link
Contributor

bors commented Dec 23, 2019

💔 Test failed - checks-travis

* ToString and AsRef are in prelude, no need to import them
@tesuji
Copy link
Contributor Author

tesuji commented Dec 23, 2019

@bors r=flip1995

@bors
Copy link
Contributor

bors commented Dec 23, 2019

📌 Commit f5b8964 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Dec 23, 2019

⌛ Testing commit f5b8964 with merge 82b0325...

bors added a commit that referenced this pull request Dec 23, 2019
a few minor cleanups

changelog: none
@bors
Copy link
Contributor

bors commented Dec 23, 2019

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

@bors bors merged commit f5b8964 into rust-lang:master Dec 23, 2019
@tesuji tesuji deleted the take branch December 23, 2019 20:44
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.

3 participants