Skip to content

Conversation

@emberian
Copy link
Contributor

@emberian emberian commented May 5, 2015

These have been deprecated for some time. This is a:

[breaking-change]

@rust-highfive
Copy link
Contributor

r? @alexcrichton

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(I figure these will come in handy in the future)

@alexcrichton
Copy link
Member

Thanks for this! At this point though we're not removing any deprecated stable functionality (we've already cleaned out what was intended to go), but in general removing deprecated unstable items is fine. I'd leave the deprecated unicode stuff for now as it moved out somewhat recently, however.

These have been deprecated for some time. This is a:

[breaking-change]
@bors
Copy link
Collaborator

bors commented May 8, 2015

☔ The latest upstream changes (presumably #25136) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton
Copy link
Member

Now that 1.0 has been released I believe that this is now a breaking change, so closing.

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.

5 participants