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

rustc: implement unstable(issue = "nnn"). #26747

Merged
merged 1 commit into from
Jul 7, 2015

Conversation

huonw
Copy link
Member

@huonw huonw commented Jul 3, 2015

This takes an issue number and points people to it in the printed error
message. This commit does not make it an error to have no issue field.

@rust-highfive
Copy link
Collaborator

r? @brson

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

@huonw
Copy link
Member Author

huonw commented Jul 3, 2015

cc @rust-lang/libs, particularly @aturon and @alexcrichton

@@ -501,6 +517,11 @@ fn find_stability_generic<'a,
either stable or unstable attribute");
}
}
} else if stab.as_ref().map_or(false, |s| s.level == Unstable && s.issue.is_none()) {
// no-deprecated unstable items need to point to issues.
Copy link
Contributor

Choose a reason for hiding this comment

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

non-deprecated

@Gankra
Copy link
Contributor

Gankra commented Jul 3, 2015

r=me for the actual code with nits addressed

Actual feature itself seems legit (I vaguely recall overhearing this discussed as desirable earlier today)

@brson
Copy link
Contributor

brson commented Jul 6, 2015

No opinion here.

@brson brson assigned Gankra and unassigned brson Jul 6, 2015
This takes an issue number and points people to it in the printed error
message. This commit does not make it an error to have no `issue` field.
@alexcrichton
Copy link
Member

@bors: r+ 69d340a

bors added a commit that referenced this pull request Jul 7, 2015
This takes an issue number and points people to it in the printed error
message. This commit does not make it an error to have no `issue` field.
@bors
Copy link
Contributor

bors commented Jul 7, 2015

⌛ Testing commit 69d340a with merge 5d53921...

@bors bors merged commit 69d340a into rust-lang:master Jul 7, 2015
@huonw huonw deleted the stability-issue branch July 16, 2015 05:30
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.

6 participants