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

Fix broken link in rustc_plugin doc #60131

Merged
merged 1 commit into from
May 6, 2019
Merged

Fix broken link in rustc_plugin doc #60131

merged 1 commit into from
May 6, 2019

Conversation

agnxy
Copy link
Contributor

@agnxy agnxy commented Apr 20, 2019

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 20, 2019
@tesuji
Copy link
Contributor

tesuji commented Apr 28, 2019

r? @Centril

@Centril
Copy link
Contributor

Centril commented Apr 28, 2019

I don't know what the policy around linking is; r? @ehuss

@rust-highfive rust-highfive assigned ehuss and unassigned steveklabnik Apr 28, 2019
@Centril
Copy link
Contributor

Centril commented Apr 29, 2019

@bors delegate=ehuss

@bors
Copy link
Contributor

bors commented Apr 29, 2019

✌️ @ehuss can now approve this pull request

@ehuss
Copy link
Contributor

ehuss commented May 4, 2019

Considering that these aren't distributed with rustup (AFAIK), and that these are only published on nightly, I'm inclined to just use the full https link. In the rare case someone is working offline, I think they'll be able to figure out what to do.

@bors r+

@bors
Copy link
Contributor

bors commented May 4, 2019

📌 Commit f77b602 has been approved by ehuss

@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 May 4, 2019
@ehuss
Copy link
Contributor

ehuss commented May 4, 2019

@bors rollup

Manishearth added a commit to Manishearth/rust that referenced this pull request May 5, 2019
bors added a commit that referenced this pull request May 5, 2019
Rollup of 5 pull requests

Successful merges:

 - #60131 (Fix broken link in rustc_plugin doc)
 - #60426 (Stop `-O`/`-C opt-level` and `-g`/`-C debuginfo` conflicting)
 - #60515 (use span instead of div for since version)
 - #60530 (rustc: rename all occurences of "freevar" to "upvar".)
 - #60536 (Correct code points to match their textual description)

Failed merges:

r? @ghost
@bors bors merged commit f77b602 into rust-lang:master May 6, 2019
@agnxy agnxy deleted the doc-link branch May 6, 2019 13:46
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.

Broken link to unstable book in rustc_plugin API docs
7 participants