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

Support future deprecation for rustc_deprecated #51681

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

varkor
Copy link
Member

@varkor varkor commented Jun 21, 2018

Follow-up to #49179 to allow since parameters to be set to future versions of Rust and correspondingly to not be treated as deprecated until that version. This is required for #30459 to be completed (though we'll need to wait until this hits beta).

@rust-highfive
Copy link
Collaborator

r? @eddyb

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 21, 2018
@varkor varkor force-pushed the rustc_deprecated-future-deprecation branch from 1a217b2 to a6d737c Compare June 21, 2018 14:59
@varkor varkor force-pushed the rustc_deprecated-future-deprecation branch from a6d737c to 0931094 Compare June 21, 2018 15:00
@rust-lang rust-lang deleted a comment from rust-highfive Jun 21, 2018
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Jun 21, 2018

📌 Commit 0931094 has been approved by petrochenkov

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 21, 2018
@petrochenkov petrochenkov assigned petrochenkov and unassigned eddyb Jun 21, 2018
@bors
Copy link
Contributor

bors commented Jun 22, 2018

⌛ Testing commit 0931094 with merge cbc4c83...

bors added a commit that referenced this pull request Jun 22, 2018
…petrochenkov

Support future deprecation for rustc_deprecated

Follow-up to #49179 to allow `since` parameters to be set to future versions of Rust and correspondingly to not be treated as deprecated until that version. This is required for #30459 to be completed (though we'll need to wait until this hits beta).
@bors
Copy link
Contributor

bors commented Jun 22, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: petrochenkov
Pushing cbc4c83 to master...

@bors bors merged commit 0931094 into rust-lang:master Jun 22, 2018
@varkor varkor deleted the rustc_deprecated-future-deprecation branch June 22, 2018 22:16
@withoutboats
Copy link
Contributor

A question of my curiosity: how does a future deprecated method appear in rustdoc? e.g. what will trim_left look like in the docs before 1.33 if we merge #52994?

@varkor
Copy link
Member Author

varkor commented Aug 8, 2018

@withoutboats: there are a couple of example screenshots in #49179! The text in them is a little outdated (the wording was changing to "Deprecating in [version]."), but the visuals are the same (I would create another screenshot, but I'm not able to right now).

@withoutboats
Copy link
Contributor

@varkor Thanks! Exactly what I was hoping it looked like :)

bors added a commit that referenced this pull request Sep 5, 2018
Add trim_start, trim_end etc.; deprecate trim_left, trim_right, etc. in future

Adds the methods: `trim_start`, `trim_end`, `trim_start_matches` and `trim_end_matches`.
Deprecates `trim_left`, `trim_right`, `trim_left_matches` and `trim_right_matches` starting from Rust 1.33.0, three versions from when they'll initially be marked as being deprecated, using the future deprecation from #30785 and #51681.

Fixes #30459.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants