Skip to content

Support future deprecation for rustc_deprecated #51681

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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
Contributor

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
Collaborator

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
Collaborator

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
Collaborator

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.
# for free to join this conversation on GitHub. Already have an account? # 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.

6 participants