Skip to content

ci(phplint): parallelize the phplint run#977

Merged
JohnVillalovos merged 1 commit intodevelopfrom
jlvillal/ci_phplint
Feb 10, 2026
Merged

ci(phplint): parallelize the phplint run#977
JohnVillalovos merged 1 commit intodevelopfrom
jlvillal/ci_phplint

Conversation

@JohnVillalovos
Copy link
Copy Markdown
Collaborator

On my system it would previously take ~46 seconds to run, now it takes ~12 seconds.

Copilot AI review requested due to automatic review settings February 7, 2026 06:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR speeds up the CI PHP syntax lint step by running php -l across files in parallel, reducing total runtime by leveraging multiple workers.

Changes:

  • Add configurable parallelism via PHP_LINT_JOBS (default 4).
  • Replace the serial per-file loop with a find ... -print0 | xargs ... php -l parallel pipeline.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci/ci-phplint
@labmecanicatec
Copy link
Copy Markdown
Collaborator

Previously 39 seconds, now 10 seconds

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/ci_phplint branch 2 times, most recently from 50f3cca to efb00c7 Compare February 10, 2026 05:11
Split the phplint job across the number of processors on the system.

On my system it would previously take ~46 seconds to run, now it takes
~12 seconds.
@JohnVillalovos JohnVillalovos enabled auto-merge (rebase) February 10, 2026 05:13
@JohnVillalovos JohnVillalovos merged commit aa0739a into develop Feb 10, 2026
11 checks passed
@JohnVillalovos JohnVillalovos deleted the jlvillal/ci_phplint branch February 10, 2026 05:14
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.

3 participants