Просмотр исходного кода

Dependent PR workflow: don't crash if `first_commit` is `null`. (#7137)

Example crash:
https://github.com/carbon-language/carbon-lang/actions/runs/25071912124/job/73454186818?pr=7122

Assisted-by: Gemini via Antigravity
Richard Smith 4 дней назад
Родитель
Сommit
7bb86bad66
1 измененных файлов с 3 добавлено и 1 удалено
  1. 3 1
      github_tools/check_dependent_pr.py

+ 3 - 1
github_tools/check_dependent_pr.py

@@ -234,7 +234,9 @@ def _parse_and_validate_state(
             parsed_merged.append(val)
     if "first_commit" in raw_state:
         fc = raw_state["first_commit"]
-        if isinstance(fc, str) and re.fullmatch(r"[0-9a-fA-F]{40}", fc):
+        if fc is None:
+            first_commit = None
+        elif isinstance(fc, str) and re.fullmatch(r"[0-9a-fA-F]{40}", fc):
             first_commit = fc
         else:
             raise ValueError(