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 cmp::Ordering::* public reexport #20140

Merged
merged 1 commit into from
Dec 23, 2014
Merged

Conversation

frewsxcv
Copy link
Member

Part of #19253

I would have removed this public reexport in #19842, but #19812 hadn't merged (and snapshotted) at the time

In #19407, I changed the codebase to stop utilizing this reexport

[breaking-change]

Part of rust-lang#19253

I would have removed this public reexport in rust-lang#19842, but rust-lang#19812 hadn't merged (and snapshotted) at the time

In rust-lang#19407, I changed the codebase to stop utilizing this reexport

[breaking-change]
@rust-highfive
Copy link
Collaborator

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 22, 2014
Part of rust-lang#19253

I would have removed this public reexport in rust-lang#19842, but rust-lang#19812 hadn't merged (and snapshotted) at the time

In rust-lang#19407, I changed the codebase to stop utilizing this reexport

[breaking-change]
@bors bors merged commit 02feaf2 into rust-lang:master Dec 23, 2014
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.

4 participants