Skip to content

ci: cancel superseded runs; skip docs PRs#440

Merged
villagestevers merged 1 commit into
villagesql:mainfrom
pyshx:pyshx/ci-concurrency-paths
May 6, 2026
Merged

ci: cancel superseded runs; skip docs PRs#440
villagestevers merged 1 commit into
villagesql:mainfrom
pyshx:pyshx/ci-concurrency-paths

Conversation

@pyshx

@pyshx pyshx commented May 6, 2026

Copy link
Copy Markdown
Contributor

Description

Two workflow tweaks aimed at the partial-miss tier (~10-13min) build that dominates everyday PR iteration:

  1. Concurrency cancellation - in-progress runs on the same PR branch are cancelled when a new push arrives. main pushes are not cancelled (cancel-in-progress is gated on github.event_name == 'pull_request') so each merged commit still gets its own verification pass.

  2. paths-ignore on the Build and Test trigger - PRs that touch only markdown files or Docs/ skip the full build/test cycle entirely. PR Validation (villint) keeps its current trigger so doc PRs still pass through linting.

If 'Build and Test VillageSQL' is configured as a required status check on main, doc-only PRs may report it as missing rather than passing - will need a no-op fallback job in that case.

Related Issue

How was this tested?

Checklist

  • I have signed the CLA
  • I have read CONTRIBUTING.md
  • I have added or updated tests as appropriate
  • My changes maintain compatibility with upstream MySQL 8.4

Two workflow tweaks aimed at the partial-miss tier (~10-13min) build
that dominates everyday PR iteration:

1. Concurrency cancellation - in-progress runs on the same PR branch
   are cancelled when a new push arrives. main pushes are not cancelled
   (cancel-in-progress is gated on github.event_name == 'pull_request')
   so each merged commit still gets its own verification pass.

2. paths-ignore on the Build and Test trigger - PRs that touch only
   markdown files or Docs/ skip the full build/test cycle entirely.
   PR Validation (villint) keeps its current trigger so doc PRs still
   pass through linting.

If 'Build and Test VillageSQL' is configured as a required status check
on main, doc-only PRs may report it as missing rather than passing -
will need a no-op fallback job in that case.
@pyshx pyshx changed the title ci: cancel superseded PR runs; skip build on docs-only PRs ci: cancel superseded runs; skip docs-only May 6, 2026
@pyshx pyshx changed the title ci: cancel superseded runs; skip docs-only ci: cancel superseded runs; skip docs May 6, 2026
@pyshx pyshx changed the title ci: cancel superseded runs; skip docs ci: cancel superseded runs; skip docs PRs May 6, 2026
@villagestevers villagestevers merged commit 220a7a8 into villagesql:main May 6, 2026
3 of 9 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants