Skip to content

ci: shorten PR check time - #538

Merged
josephfusco merged 3 commits into
mainfrom
ci/faster-pr-checks
Nov 1, 2026
Merged

josephfusco merged 3 commits into
mainfrom
ci/faster-pr-checks

Conversation

@josephfusco

Copy link
Copy Markdown
Member

Shortens the two jobs that decide how long a PR waits for checks. On recent human PRs, Schema Linter (median 101s) and Test Plugin (median 98s) were the last to finish on almost every commit.

  • Test Plugin / Test Plugin (Gutenberg): start MySQL in the background before the image build, so its pull (~11s) and first boot (~8s) overlap the build instead of following it.
  • Schema Linter: install graphql-inspector globally alongside the linter. The old local npm install in the repo root also installed @wordpress/scripts from the lockfile (~23s). The Docker-based action-get-latest-tag, rebuilt every run, is replaced with gh release view.
Use of AI Tools

AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: Assisting with analyzing CI run history and writing the workflow changes.

@josephfusco
josephfusco marked this pull request as ready for review November 1, 2026 01:52
@josephfusco
josephfusco requested review from a team and colinmurphy as code owners November 1, 2026 01:52
@josephfusco
josephfusco requested a balanced review from Copilot November 1, 2026 01:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The workflow can finish successfully without a baseline schema because the release lookup failure is masked and subsequent download and inspector failures do not stop execution.

1 open issue
What changed in this PR

This PR aims to reduce PR check times by overlapping database startup with image builds and removing unnecessary setup work from schema validation.

Changes:

  • Start MySQL in the background in both plugin test workflows, then wait for it before starting WordPress.
  • Install GraphQL Inspector globally alongside the schema linter to avoid installing repository development dependencies.
  • Replace the Docker-based latest-tag action with gh release view.
File Description
.github/​workflows/​test-plugin.yml Overlaps database startup with the test image build.
.github/​workflows/​test-plugin-gutenberg.yml Applies the same startup optimization to Gutenberg tests.
.github/​workflows/​schema-linter.yml Streamlines GraphQL tooling installation and release lookup.

Review effort: Balanced

Comment thread .github/workflows/schema-linter.yml Resolved
…ng up the baseline release tag'

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@josephfusco
josephfusco merged commit d7fbb16 into main Nov 1, 2026
17 checks passed
@josephfusco
josephfusco deleted the ci/faster-pr-checks branch November 1, 2026 03:31
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.

2 participants