Skip to content

Stabilize tooling CI commands#92

Merged
mehul0810 merged 6 commits into
release/1.6.0from
fix/release-1.6.0-tooling-ci-readiness
May 21, 2026
Merged

Stabilize tooling CI commands#92
mehul0810 merged 6 commits into
release/1.6.0from
fix/release-1.6.0-tooling-ci-readiness

Conversation

@mehul0810
Copy link
Copy Markdown
Collaborator

@mehul0810 mehul0810 commented May 21, 2026

Summary

  • Adds the missing dev dependencies used by the existing Composer lint and test scripts, and switches those scripts to vendor/bin executables.
  • Replaces the unresolved checkstyle annotation action usage with the supported cs2pr CLI installed by setup-php.
  • Pins Composer platform resolution to PHP 8.1 so the lock file remains installable across the release CI matrix.
  • Scopes PHPStan to plugin PHP entry points and src, adds a runtime bootstrap, and records the current baseline so new static-analysis errors are gated.
  • Removes stale PHPUnit XML entries so the existing test suite runs without configuration warnings.
  • Runs code-style CI against changed PHP files so release PRs are gated without forcing a full legacy formatting sweep.

Validation

  • composer validate --no-check-all
  • composer install --dry-run --no-scripts --no-interaction
  • composer lint
  • composer test
  • composer phpstan
  • ./vendor/bin/phpcs -s --no-cache --report-full phpstan-bootstrap.php
  • git diff --check

Remaining

  • composer check-cs now runs with a default target, but still fails on existing repo-wide PHPCS debt. This PR keeps full cleanup tracked by Pass Linting CI/CD #53 while making CI useful for changed PHP files.

Refs #53.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Test on Playground
Test this pull request on the Playground

@mehul0810 mehul0810 force-pushed the fix/release-1.6.0-tooling-ci-readiness branch from b98a83e to e1fc9c9 Compare May 21, 2026 06:43
@mehul0810 mehul0810 merged commit fd89cc0 into release/1.6.0 May 21, 2026
6 checks passed
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