Skip to content

GitHub Actions: Use new PHP workflow#1251

Merged
lippserd merged 4 commits into
mainfrom
php-workflow
Jan 16, 2026
Merged

GitHub Actions: Use new PHP workflow#1251
lippserd merged 4 commits into
mainfrom
php-workflow

Conversation

@JolienTrog

@JolienTrog JolienTrog commented Jul 11, 2025

Copy link
Copy Markdown

This PR switches to the new PHP workflow, which combines linting, static analysis, and unit testing. phpcs.xml and phpunit.xml have been removed, as the new workflow provides sane defaults for both and the files did not contain any special configurations anyway. In addition, code style corrections were made to comply with the latest PSR coding standards. References to removed PHP files have been cleaned up from the phpstan baseline.

  • After merging, @lippserd will update the branch protection required status checks to match the new workflow.

@JolienTrog JolienTrog requested a review from lippserd July 11, 2025 12:29
@cla-bot cla-bot Bot added the cla/signed CLA is signed by all contributors of a PR label Jul 11, 2025
@sukhwinder33445 sukhwinder33445 changed the title Use reusable PHP test workflow for icinga/github-actions GitHub Actions: Use new PHP workflow Dec 15, 2025
@sukhwinder33445 sukhwinder33445 self-assigned this Dec 15, 2025
- Remove obsolete workflow trigger for `release/*` branch pushes
- Run workflow on push to branch `support/*`
Use defaults from new workflows.
@sukhwinder33445 sukhwinder33445 requested review from lippserd and removed request for lippserd January 15, 2026 08:02

@sukhwinder33445 sukhwinder33445 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.

It's ready, please check it out @lippserd.

@lippserd lippserd merged commit bf431dc into main Jan 16, 2026
13 checks passed
@lippserd lippserd deleted the php-workflow branch January 16, 2026 08:06
@lippserd lippserd mentioned this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed CLA is signed by all contributors of a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants