Skip to content

ci: merge the cheap script gates into one job; cancel superseded bench runs - #6204

Merged
shai-almog merged 1 commit into
masterfrom
ci-merge-source-gates
Nov 10, 2026
Merged

shai-almog merged 1 commit into
masterfrom
ci-merge-source-gates

Conversation

@shai-almog

Copy link
Copy Markdown
Collaborator

Six tree-wide script gates (control characters, author tags, since tags, GMS isolation, 16 KB page alignment, copyright headers) each ran as their own workflow, so on a busy PR each queued for a runner of its own. A sub-minute grep showed 40+ minutes of wall time, nearly all of it waiting.

  • They now run as steps of one job in source-gates.yml, with no paths filter (so a scripts/**-only PR still runs them) and if: !cancelled() on each step so one failure does not hide the others.
  • python-bench.yml and build-container.yml gain cancel-in-progress. Reusable workflows are left alone (github.workflow resolves to the caller, so a group there would cancel the caller), as are release/publish/scheduled jobs.
  • References in CLAUDE.md and check-android-dependency-alignment.yml updated.

actionlint (scripts/check-workflows.sh) passes.

🤖 Generated with Claude Code

…h runs

Six tree-wide script gates each queued for a runner of their own; on a busy
PR a sub-minute grep showed 40+ minutes of wall time, nearly all waiting.
They now run as steps of one job in source-gates.yml, still with no paths
filter. python-bench and build-container gain cancel-in-progress.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

✅ Continuous Quality Report

Test & Coverage

Static Analysis

  • SpotBugs [Report archive]
    • ✅ ByteCodeTranslator: 0 findings (no issues)
    • ✅ android: 0 findings (no issues)
    • ✅ android-compat: 0 findings (no issues)
    • ✅ android-res-compiler: 0 findings (no issues)
    • ✅ backend: 0 findings (no issues)
    • ✅ backend-test: 0 findings (no issues)
    • ✅ build-engine: 0 findings (no issues)
    • ✅ build-hint-catalog: 0 findings (no issues)
    • ✅ build-hint-tools: 0 findings (no issues)
    • ✅ codenameone-gradle-plugin: 0 findings (no issues)
    • ✅ codenameone-maven-plugin: 0 findings (no issues)
    • ✅ compose-compat: 0 findings (no issues)
    • ✅ core-unittests: 0 findings (no issues)
    • ✅ desktop-compat-runtime: 0 findings (no issues)
    • ✅ fxml-compiler: 0 findings (no issues)
    • ✅ ios: 0 findings (no issues)
    • ✅ javac: 0 findings (no issues)
    • ✅ javafx-compat: 0 findings (no issues)
    • ✅ project-model: 0 findings (no issues)
    • ✅ python-runtime: 0 findings (no issues)
    • ✅ python-transpiler: 0 findings (no issues)
    • ✅ swing-compat: 0 findings (no issues)
  • ✅ PMD: 0 findings (no issues) [Report archive]
  • ✅ Checkstyle: 0 findings (no issues) [Report archive]

Generated automatically by the PR CI workflow.

@shai-almog
shai-almog merged commit e4a3718 into master Nov 10, 2026
18 checks passed
@shai-almog
shai-almog deleted the ci-merge-source-gates branch November 10, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant