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

Update reference link in doc's 404 #45778

Merged
merged 1 commit into from
Nov 7, 2017
Merged

Update reference link in doc's 404 #45778

merged 1 commit into from
Nov 7, 2017

Conversation

Havvy
Copy link
Contributor

@Havvy Havvy commented Nov 5, 2017

It's currently linking to a page that says it's on the page I'm changing the link too.

It's currently linking to a page that says it's on the page I'm changing the link too.
@rust-highfive
Copy link
Collaborator

r? @frewsxcv

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

@QuietMisdreavus
Copy link
Member

QuietMisdreavus commented Nov 5, 2017

Good catch! r=me pending travis.

(EDIT: though i'm not sure if linkchecker goes through all these pages? eh, habit *shrug*)

@kennytm kennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Nov 5, 2017
@kennytm
Copy link
Member

kennytm commented Nov 5, 2017

@bors r=QuietMisdreavus rollup

@bors
Copy link
Contributor

bors commented Nov 5, 2017

📌 Commit 6363b06 has been approved by QuietMisdreavus

@steveklabnik
Copy link
Member

@bors: r+ rollup

thanks! i wonder why linkchecker isn't run here...

@bors
Copy link
Contributor

bors commented Nov 5, 2017

💡 This pull request was already approved, no need to approve it again.

@bors
Copy link
Contributor

bors commented Nov 5, 2017

📌 Commit 6363b06 has been approved by steveklabnik

@kennytm
Copy link
Member

kennytm commented Nov 5, 2017

I don't think the linkchecker can verify if an external link is valid...

@Havvy
Copy link
Contributor Author

Havvy commented Nov 5, 2017

The prior link is still valid insomuch as there's a page there. It's just not a useful page.

@frewsxcv frewsxcv removed their assignment Nov 7, 2017
kennytm added a commit to kennytm/rust that referenced this pull request Nov 7, 2017
Update reference link in doc's 404

It's currently linking to a page that says it's on the page I'm changing the link too.
kennytm added a commit to kennytm/rust that referenced this pull request Nov 7, 2017
Update reference link in doc's 404

It's currently linking to a page that says it's on the page I'm changing the link too.
bors added a commit that referenced this pull request Nov 7, 2017
Rollup of 9 pull requests

- Successful merges: #45470, #45588, #45682, #45714, #45751, #45764, #45778, #45782, #45784
- Failed merges:
@bors bors merged commit 6363b06 into rust-lang:master Nov 7, 2017
@Havvy Havvy deleted the patch-1 branch November 7, 2017 21:11
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.

7 participants