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

Remove unnecessary pub mod #1116

Merged
merged 1 commit into from
May 18, 2017
Merged

Remove unnecessary pub mod #1116

merged 1 commit into from
May 18, 2017

Conversation

mattico
Copy link
Contributor

@mattico mattico commented May 12, 2017

rust-lang/rust#18241 is fixed now, so this mod doesn't need to be pub.

Possible downside: the minimum compiler version to build is now 1.9.0 or thereabouts.

rust-lang/rust#18241 is fixed now, so this mod doesn't need to be pub.
@Diggsey
Copy link
Contributor

Diggsey commented May 17, 2017

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented May 17, 2017

📌 Commit fc563b5 has been approved by Diggsey

@bors
Copy link
Contributor

bors commented May 18, 2017

⌛ Testing commit fc563b5 with merge b2f39ba...

bors added a commit that referenced this pull request May 18, 2017
Remove unnecessary pub mod

rust-lang/rust#18241 is fixed now, so this mod doesn't need to be pub.

Possible downside: the minimum compiler version to build is now 1.9.0 or thereabouts.
@bors
Copy link
Contributor

bors commented May 18, 2017

💔 Test failed - status-appveyor

@Diggsey
Copy link
Contributor

Diggsey commented May 18, 2017

@bors retry

@bors
Copy link
Contributor

bors commented May 18, 2017

⌛ Testing commit fc563b5 with merge 0ad45aa...

bors added a commit that referenced this pull request May 18, 2017
Remove unnecessary pub mod

rust-lang/rust#18241 is fixed now, so this mod doesn't need to be pub.

Possible downside: the minimum compiler version to build is now 1.9.0 or thereabouts.
@bors
Copy link
Contributor

bors commented May 18, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: Diggsey
Pushing 0ad45aa to master...

@bors bors merged commit fc563b5 into rust-lang:master May 18, 2017
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