Skip to content

Fix links for typeck diagnostics without tripping tidy #43075

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

Merged
merged 3 commits into from
Jul 8, 2017

Conversation

durka
Copy link
Contributor

@durka durka commented Jul 5, 2017

Pick up #42624 to fix #43070 (cc @frewsxcv). I fixed a few other links while I was at it.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@alexcrichton alexcrichton added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 6, 2017
@nikomatsakis
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 6, 2017

📌 Commit 9b76300 has been approved by nikomatsakis

@bors
Copy link
Collaborator

bors commented Jul 6, 2017

🔒 Merge conflict

frewsxcv and others added 3 commits July 6, 2017 20:13
It used to point to the Book, but no specific section, and in fact the
information is not in the Book (see rust-lang/reference#77).
@durka durka force-pushed the error-long-link branch from 9b76300 to 7386288 Compare July 6, 2017 20:14
@durka
Copy link
Contributor Author

durka commented Jul 6, 2017

Rebased (I don't know why bors and github disagreed on whether the PR could be merged??)

@nikomatsakis
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jul 7, 2017

📌 Commit 7386288 has been approved by nikomatsakis

@bors
Copy link
Collaborator

bors commented Jul 8, 2017

⌛ Testing commit 7386288 with merge 720c596...

bors added a commit that referenced this pull request Jul 8, 2017
Fix links for typeck diagnostics without tripping tidy

Pick up #42624 to fix #43070 (cc @frewsxcv). I fixed a few other links while I was at it.
@bors
Copy link
Collaborator

bors commented Jul 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: nikomatsakis
Pushing 720c596 to master...

@bors bors merged commit 7386288 into rust-lang:master Jul 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken links to the reference from error messages
6 participants