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

add the issue number to doc_masked's feature gate #45024

Merged
merged 1 commit into from Oct 5, 2017

Conversation

Projects
None yet
4 participants
@QuietMisdreavus
Member

QuietMisdreavus commented Oct 4, 2017

Whoops, missed this in the original #[doc(masked)] PR.

@rust-highfive

This comment has been minimized.

Show comment
Hide comment
@rust-highfive

rust-highfive Oct 4, 2017

Collaborator

r? @pnkfelix

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

Collaborator

rust-highfive commented Oct 4, 2017

r? @pnkfelix

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

@pnkfelix

This comment has been minimized.

Show comment
Hide comment
@pnkfelix

pnkfelix Oct 5, 2017

Member

(Just for hyperlinking purposes, the issue being linked is #44027 )

Member

pnkfelix commented Oct 5, 2017

(Just for hyperlinking purposes, the issue being linked is #44027 )

@pnkfelix

This comment has been minimized.

Show comment
Hide comment
@pnkfelix

pnkfelix Oct 5, 2017

Member

@bors r+ rollup

Member

pnkfelix commented Oct 5, 2017

@bors r+ rollup

@bors

This comment has been minimized.

Show comment
Hide comment
@bors

bors Oct 5, 2017

Contributor

📌 Commit 2af12b7 has been approved by pnkfelix

Contributor

bors commented Oct 5, 2017

📌 Commit 2af12b7 has been approved by pnkfelix

kennytm added a commit to kennytm/rust that referenced this pull request Oct 5, 2017

Rollup merge of rust-lang#45024 - QuietMisdreavus:doc-masked-issue-nu…
…m, r=pnkfelix

add the issue number to doc_masked's feature gate

Whoops, missed this in the original `#[doc(masked)]` PR.

@kennytm kennytm referenced this pull request Oct 5, 2017

Merged

Rollup of 9 pull requests #45046

bors added a commit that referenced this pull request Oct 5, 2017

Auto merge of #45046 - kennytm:rollup, r=kennytm
Rollup of 9 pull requests

- Successful merges: #44664, #44935, #44972, #44980, #44987, #44997, #45006, #45017, #45024
- Failed merges:

@bors bors merged commit 2af12b7 into rust-lang:master Oct 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@QuietMisdreavus QuietMisdreavus deleted the QuietMisdreavus:doc-masked-issue-num branch Feb 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment