pw_presubmit: Allow Bazel issues in TODOs

Changes the pw_presubmit TODO checks to allow Bazel GitHub issues.

Change-Id: Idbdc6f90cd656ec3ac3be3de95091d385a2e8b6b
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/190970
Reviewed-by: Ted Pudlik <[email protected]>
Commit-Queue: Auto-Submit <[email protected]>
Pigweed-Auto-Submit: Armando Montanez <[email protected]>

pw_presubmit/py/pw_presubmit/todo_check.py

@@ -41,10 +41,12 @@
 )
 
 # todo-check: disable
+# pylint: disable=line-too-long
 BUGS_ONLY = re.compile(
     r'(?:\bTODO\(b/\d+(?:, ?b/\d+)*\).*\w)|'
     r'(?:\bTODO: b/\d+(?:, ?b/\d+)* - )|'
-    r'(?:\bTODO: https://issues.pigweed.dev/issues/\d+ - )'
+    r'(?:\bTODO: https://issues.pigweed.dev/issues/\d+ - )|'
+    r'(?:\bTODO: https://github\.com/bazelbuild/[a-z][-_a-z0-9]*/issues/\d+[ ]-[ ])'
 )
 BUGS_OR_USERNAMES = re.compile(
     r"""
@@ -78,11 +80,19 @@
         (?:,[ ]?(?:fxbug\.dev/\d+|[a-z]+))*  # Additional usernames or bugs.
     \)
 .*\w  # Explanation.
+)|
+(?:  # Bazel GitHub issues. No usernames allowed.
+    \bTODO:[ ]
+    (?:
+        https://github\.com/bazelbuild/[a-z][-_a-z0-9]*/issues/\d+
+    )
+[ ]-[ ].*\w  # Explanation.
 )
     """,
     re.VERBOSE,
 )
 _TODO = re.compile(r'\bTODO\b')
+# pylint: enable=line-too-long
 # todo-check: enable
 
 # If seen, ignore this line and the next.

pw_presubmit/py/todo_check_test.py

@@ -286,6 +286,26 @@ def test_fuchsia_two_bugs(self) -> None:
         self._run_bugs_users('TODO(fxbug.dev/123,fxbug.dev/321): bar\n')
         self.ctx.fail.assert_not_called()
 
+    def test_bazel_gh_issue(self) -> None:
+        contents = (
+            'TODO: https://github.com/bazelbuild/bazel/issues/12345 - '
+            'Bazel sometimes works\n'
+        )
+        self._run_bugs_users(contents)
+        self.ctx.fail.assert_not_called()
+        self._run_bugs(contents)
+        self.ctx.fail.assert_not_called()
+
+    def test_bazel_gh_issue_underscore(self) -> None:
+        contents = (
+            'TODO: https://github.com/bazelbuild/rules_cc/issues/678910 - '
+            'Sometimes it does not work\n'
+        )
+        self._run_bugs_users(contents)
+        self.ctx.fail.assert_not_called()
+        self._run_bugs(contents)
+        self.ctx.fail.assert_not_called()
+
 
 if __name__ == '__main__':
     unittest.main()