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

Issue 54246 #54299

Merged
merged 2 commits into from
Sep 18, 2018
Merged

Issue 54246 #54299

merged 2 commits into from
Sep 18, 2018

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
Collaborator

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
Member

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
Member

varkor commented Sep 17, 2018

@bors r+ rollup

Thanks!

@bors
Copy link
Contributor

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