Skip to content

Conversation

illfygli
Copy link
Contributor

I added the option of providing a help message for deprecated features, that takes precedence over the default help: remove this attribute message, along with messages for the features that mention replacements in the reason for deprecation.

Fixes #54246.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(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 Sep 17, 2018
@varkor
Copy link
Contributor

varkor commented Sep 17, 2018

Thanks, this looks good! I'll approve once everything passes on Travis!

r? @varkor

@rust-highfive rust-highfive assigned varkor and unassigned nikomatsakis Sep 17, 2018
@varkor
Copy link
Contributor

varkor commented Sep 17, 2018

@bors r+ rollup

Thanks!

@bors
Copy link
Collaborator

bors commented Sep 17, 2018

📌 Commit 79da7a0 has been approved by varkor

@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 Sep 17, 2018
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Sep 18, 2018
Issue 54246

I added the option of providing a help message for deprecated features, that takes precedence over the default `help: remove this attribute` message, along with messages for the features that mention replacements in the reason for deprecation.

Fixes rust-lang#54246.
bors added a commit that referenced this pull request Sep 18, 2018
Rollup of 9 pull requests

Successful merges:

 - #53522 (Add doc for impl From for Addr)
 - #54097 (rustdoc: Remove namespace for keywords)
 - #54205 (Add treat-err-as-bug flag in rustdoc)
 - #54225 (Regression test for #53675.)
 - #54232 (add `-Z dont-buffer-diagnostics`)
 - #54273 (Suggest to change numeric literal instead of casting)
 - #54299 (Issue 54246)
 - #54311 (Remove README with now-out-of-date docs about docs.)
 - #54313 (OsStr: Document that it's not NUL terminated)

Failed merges:

r? @ghost
@bors bors merged commit 79da7a0 into rust-lang:master Sep 18, 2018
@illfygli illfygli deleted the issue-54246 branch October 23, 2018 19:03
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.

5 participants