Skip to content

Commit

Permalink
Rollup merge of rust-lang#102017 - est31:rustc_dev_guide_tidy_skip, r…
Browse files Browse the repository at this point in the history
…=Mark-Simulacrum

Add all submodules to the list of directories tidy skips

Tidy contains a blacklist of directories that it is not visiting. This list is also used by the `replace-version-placeholder` tool added by rust-lang#100591 , to determine the directories to do its replacement from. Generally, tidy does not check submodules, but this is not done consistently for all submodules. This PR adds the submodules that were previously missing, so that the `replace-version-placeholder` tool does not attempt to change content of the books. This was needed because `rustc-dev-guide` contains the placeholder, leading to rust-lang#102014.

Fixes rust-lang#102014
  • Loading branch information
matthiaskrgr authored Sep 19, 2022
2 parents 548c3ea + cd3979d commit ade6505
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions src/tools/tidy/src/walk.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,12 @@ pub fn filter_dirs(path: &Path) -> bool {
"src/tools/rust-installer",
"src/tools/rustfmt",
"src/doc/book",
"src/doc/edition-guide",
"src/doc/embedded-book",
"src/doc/nomicon",
"src/doc/rust-by-example",
"src/doc/rustc-dev-guide",
"src/doc/reference",
// Filter RLS output directories
"target/rls",
"src/bootstrap/target",
Expand Down

0 comments on commit ade6505

Please sign in to comment.