static_checks: Fix broken link
Change-Id: I88f053b54c87547f1d33cd718a6e28aaa2b64b8e
Reviewed-on: https://pigweed-review.googlesource.com/c/infra/recipes/+/192290
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Anthony DiGirolamo <tonymd@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Rob Mohr <mohrr@google.com>
diff --git a/recipes/static_checks.expected/requires.first-paragraph.json b/recipes/static_checks.expected/requires.first-paragraph.json
index 75dbc23..100623e 100644
--- a/recipes/static_checks.expected/requires.first-paragraph.json
+++ b/recipes/static_checks.expected/requires.first-paragraph.json
@@ -274,7 +274,7 @@
"name": "check requires.warning",
"~followup_annotations": [
"@@@STEP_NEST_LEVEL@1@@@",
- "@@@STEP_SUMMARY_TEXT@Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.@@@"
+ "@@@STEP_SUMMARY_TEXT@Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.@@@"
]
},
{
@@ -284,7 +284,7 @@
"-host",
"https://pigweed-review.googlesource.com",
"-input",
- "{\"change_id\": \"123456\", \"input\": {\"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.\", \"notify\": \"OWNER\"}, \"revision_id\": \"HASH\"}",
+ "{\"change_id\": \"123456\", \"input\": {\"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.\", \"notify\": \"OWNER\"}, \"revision_id\": \"HASH\"}",
"-output",
"/path/to/tmp/json"
],
@@ -311,7 +311,7 @@
"@@@STEP_LOG_LINE@json.input@{@@@",
"@@@STEP_LOG_LINE@json.input@ \"change_id\": \"123456\",@@@",
"@@@STEP_LOG_LINE@json.input@ \"input\": {@@@",
- "@@@STEP_LOG_LINE@json.input@ \"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.\",@@@",
+ "@@@STEP_LOG_LINE@json.input@ \"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.\",@@@",
"@@@STEP_LOG_LINE@json.input@ \"notify\": \"OWNER\"@@@",
"@@@STEP_LOG_LINE@json.input@ },@@@",
"@@@STEP_LOG_LINE@json.input@ \"revision_id\": \"HASH\"@@@",
@@ -323,7 +323,7 @@
{
"failure": {
"failure": {},
- "humanReason": "Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process."
+ "humanReason": "Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process."
},
"name": "$result"
}
diff --git a/recipes/static_checks.expected/requires.middle-paragraph.json b/recipes/static_checks.expected/requires.middle-paragraph.json
index 1b9dcca..530a014 100644
--- a/recipes/static_checks.expected/requires.middle-paragraph.json
+++ b/recipes/static_checks.expected/requires.middle-paragraph.json
@@ -274,7 +274,7 @@
"name": "check requires.warning",
"~followup_annotations": [
"@@@STEP_NEST_LEVEL@1@@@",
- "@@@STEP_SUMMARY_TEXT@Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.@@@"
+ "@@@STEP_SUMMARY_TEXT@Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.@@@"
]
},
{
@@ -284,7 +284,7 @@
"-host",
"https://pigweed-review.googlesource.com",
"-input",
- "{\"change_id\": \"123456\", \"input\": {\"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.\", \"notify\": \"OWNER\"}, \"revision_id\": \"HASH\"}",
+ "{\"change_id\": \"123456\", \"input\": {\"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.\", \"notify\": \"OWNER\"}, \"revision_id\": \"HASH\"}",
"-output",
"/path/to/tmp/json"
],
@@ -311,7 +311,7 @@
"@@@STEP_LOG_LINE@json.input@{@@@",
"@@@STEP_LOG_LINE@json.input@ \"change_id\": \"123456\",@@@",
"@@@STEP_LOG_LINE@json.input@ \"input\": {@@@",
- "@@@STEP_LOG_LINE@json.input@ \"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process.\",@@@",
+ "@@@STEP_LOG_LINE@json.input@ \"message\": \"Found \\\"Requires:\\\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process.\",@@@",
"@@@STEP_LOG_LINE@json.input@ \"notify\": \"OWNER\"@@@",
"@@@STEP_LOG_LINE@json.input@ },@@@",
"@@@STEP_LOG_LINE@json.input@ \"revision_id\": \"HASH\"@@@",
@@ -323,7 +323,7 @@
{
"failure": {
"failure": {},
- "humanReason": "Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for the new dependent CL process."
+ "humanReason": "Found \"Requires:\" line 'Change-Id: I1' but they are not allowed. See https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes for the new dependent CL process."
},
"name": "$result"
}
diff --git a/recipes/static_checks.py b/recipes/static_checks.py
index 3430c1a..8af6724 100644
--- a/recipes/static_checks.py
+++ b/recipes/static_checks.py
@@ -135,8 +135,8 @@
if not allow_requires and has_requires_line:
_warn(
'Found "Requires:" line {!r} but they are not allowed. See '
- 'https://pigweed.dev/docs/ci_cq_intro.html#dependent-changes for '
- 'the new dependent CL process.'.format(line)
+ 'https://pigweed.dev/docs/infra/ci_cq_intro.html#dependent-changes '
+ 'for the new dependent CL process.'.format(line)
)
all_warnings: str = '\n\n'.join(warnings)
