Apply suggestions from code review

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

Author: sethmlarson

.github/workflows/build.yml

@@ -680,7 +680,6 @@ jobs:
     with:
       oss-fuzz-project-name: ${{ matrix.oss-fuzz-project-name }}
       sanitizer: ${{ matrix.sanitizer }}
-      timeout-minutes: 60
 
   all-required-green:  # This job does nothing and is only used for the branch protection
     name: All required checks pass
@@ -736,10 +735,8 @@ jobs:
           }}
           ${{ !fromJSON(needs.build-context.outputs.run-windows-tests) && 'build-windows,' || '' }}
           ${{
-            (
-              !fromJSON(needs.build-context.outputs.run-ci-fuzz)
-              && !fromJSON(needs.build-context.outputs.run-ci-fuzz-stdlib)
-            )
+            !fromJSON(needs.build-context.outputs.run-ci-fuzz)
+            && !fromJSON(needs.build-context.outputs.run-ci-fuzz-stdlib)
             && 'cifuzz,' ||
             ''
           }}

.github/workflows/reusable-cifuzz.yml

@@ -12,19 +12,12 @@ on:
         description: OSS-Fuzz sanitizer
         required: true
         type: string
-      timeout-minutes:
-        description: Timeout in minutes for the action
-        required: false
-        default: 60
-        type: number
 
 jobs:
   cifuzz:
-    name: >-
-      ${{ inputs.oss-fuzz-project-name }}
-      (${{ inputs.sanitizer }})
+    name: ${{ inputs.oss-fuzz-project-name }} (${{ inputs.sanitizer }})
     runs-on: ubuntu-latest
-    timeout-minutes: ${{ inputs.timeout-minutes }}
+    timeout-minutes: 60
     steps:
       - name: Build fuzzers (${{ inputs.sanitizer }})
         id: build

Tools/build/compute-changes.py

@@ -145,9 +145,9 @@ def compute_changes() -> None:
         print("Branch too old for CIFuzz tests; or no C files were changed")
 
     if outputs.run_ci_fuzz_stdlib:
-        print("Run CIFuzz tests for libraries")
+        print("Run CIFuzz tests for stdlib")
     else:
-        print("Branch too old for CIFuzz tests; or no library files were changed")
+        print("Branch too old for CIFuzz tests; or no stdlib files were changed")
 
     if outputs.run_docs:
         print("Build documentation")
@@ -224,7 +224,7 @@ def process_changed_files(changed_files: Set[Path]) -> Outputs:
         doc_file = file.suffix in SUFFIXES_DOCUMENTATION or doc_or_misc
 
         if file.parent == GITHUB_WORKFLOWS_PATH:
-            if file.name == "build.yml" or file.name == "reusable-cifuzz.yml":
+            if file.name in ("build.yml", "reusable-cifuzz.yml"):
                 run_tests = run_ci_fuzz = run_ci_fuzz_stdlib = True
                 has_platform_specific_change = False
             if file.name == "reusable-docs.yml":