Skip to content

Commit 2eba3af

Browse files
authored
Rollup merge of #142666 - jieyouxu:skip-triagebot-check, r=Kobzol
Skip tidy triagebot linkcheck if `triagebot.toml` doesn't exist Since distribution tarballs won't include `triagebot.toml`. I think it's sufficiently obvious if `triagebot.toml` gets deleted entirely in PRs. r? Kobzol
2 parents c65751c + 4b50703 commit 2eba3af

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/tools/tidy/src/triagebot.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,11 @@ use toml::Value;
66

77
pub fn check(path: &Path, bad: &mut bool) {
88
let triagebot_path = path.join("triagebot.toml");
9+
10+
// This check is mostly to catch broken path filters *within* `triagebot.toml`, and not enforce
11+
// the existence of `triagebot.toml` itself (which is more obvious), as distribution tarballs
12+
// will not include non-essential bits like `triagebot.toml`.
913
if !triagebot_path.exists() {
10-
tidy_error!(bad, "triagebot.toml file not found");
1114
return;
1215
}
1316

0 commit comments

Comments
 (0)