Skip to content

Feat/php ci workflows#14

Merged
orangit-sami-bister merged 2 commits into
mainfrom
feat/php-ci-workflows
Apr 30, 2026
Merged

Feat/php ci workflows#14
orangit-sami-bister merged 2 commits into
mainfrom
feat/php-ci-workflows

Conversation

@spexii
Copy link
Copy Markdown
Contributor

@spexii spexii commented Apr 30, 2026

Description

Adds PHP / Symfony lint and CI pipeline support to platform-tooling, bringing PHP closer to parity with the existing Node and Python coverage.

New reusable workflows:

  • php-lint: runs PHP_CodeSniffer and PHPStan in parallel, plus hadolint for Docker lint
  • php-test: installs Composer dependencies and runs PHPUnit (unit tests only)
  • php-ci: full CI pipeline: security -> lint + test (parallel) -> vulnerability scan
  • php-daily: nightly pipeline: full CI + OWASP ZAP DAST scan in parallel

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation update
  • CI/CD or tooling change

How has this been tested?

  • Unit tests
  • Integration tests
  • Manual testing

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have added/updated tests that prove my fix or feature works
  • I have updated the documentation accordingly
  • My changes do not introduce new warnings or errors
  • Any dependent changes have been merged and published

Screenshots / recordings (if applicable)

N/A

Additional context

  • phpcs and phpstan must be installed as Composer dev dependencies in the consuming project (vendor/bin/phpcs and vendor/bin/phpstan). The workflow fails immediately with a clear error if either binary is missing.
  • If a project has phpcs.xml or phpstan.neon committed, those config files take precedence over all workflow inputs
  • After merge, all # pt-sha references across all workflow files must be updated to the new merge commit SHA using the bump process in the README

Copy link
Copy Markdown
Contributor

@orangit-sami-bister orangit-sami-bister left a comment

Choose a reason for hiding this comment

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

It is consistent with other ones. Looks good. great jib

@orangit-sami-bister orangit-sami-bister merged commit f036f3b into main Apr 30, 2026
2 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.

2 participants