Skip to content

Enforce dashes in the unstable book file names #47414

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 15, 2018
Merged

Conversation

est31
Copy link
Member

@est31 est31 commented Jan 13, 2018

Also rename the existing underscore using files to use dashes.

Fixes #47394.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@Mark-Simulacrum Mark-Simulacrum added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 13, 2018
@Mark-Simulacrum
Copy link
Member

r? @alexcrichton

Looks like there's some compilation failures: https://travis-ci.org/rust-lang/rust/builds/328459014#L1396.

Also rename the existing underscore using files to use dashes.

Fixes rust-lang#47394.
@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Jan 13, 2018

📌 Commit 38e2667 has been approved by alexcrichton

@estebank estebank 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 Jan 14, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Jan 15, 2018
Enforce dashes in the unstable book file names

Also rename the existing underscore using files to use dashes.

Fixes rust-lang#47394.
bors added a commit that referenced this pull request Jan 15, 2018
Rollup of 10 pull requests

- Successful merges: #47120, #47126, #47277, #47330, #47368, #47372, #47414, #47417, #47432, #47443
- Failed merges: #47334
@bors bors merged commit 38e2667 into rust-lang:master Jan 15, 2018
est31 added a commit to est31/rust that referenced this pull request Jan 15, 2018
Previously, any deadlinks from a book's SUMMARY.md wouldn't
cause any errors or warnings or similar but mdbook would simply
create a page with blank content.

This has kept bug rust-lang#47394 hidden. It should have been detected
back in the PR when those wrongly named files got added to the
book.

PR rust-lang#47414 was one component of the solution. This change
is a second line of defense for the unstable book and a first
line of defense for any other book.

We also update mdbook to the most recent version.
kennytm added a commit to kennytm/rust that referenced this pull request Jan 23, 2018
…hton

Check for deadlinks from the summary during book generation

Previously, any deadlinks from a book's SUMMARY.md wouldn't
cause any errors or warnings or similar but mdbook would simply
create a page with blank content.

This has kept bug rust-lang#47394 hidden. It should have been detected
back in the PR when those wrongly named files got added to the
book.

PR rust-lang#47414 was one component of the solution. This change
is a second line of defense for the unstable book and a first
line of defense for any other book.

We also update mdbook to the most recent version.
# for free to join this conversation on GitHub. Already have an account? # 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