Skip to content

fix: Correct php-cs-fixer setup - #938

Merged
DannyvdSluijs merged 2 commits into
mainfrom
fix/correct-phpcs-paths
Aug 21, 2026
Merged

fix: Correct php-cs-fixer setup#938
DannyvdSluijs merged 2 commits into
mainfrom
fix/correct-phpcs-paths

Conversation

@DannyvdSluijs

@DannyvdSluijs DannyvdSluijs commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Description

Code style was missing the demo folder and the CI/CD workflow was overriding the paths to check

Related Issue

Fixes #935

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING guidelines
  • My code follows the code style of this project
  • I have added tests that prove my fix is effective or that my feature works
  • All new and existing tests pass
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional Notes

@DannyvdSluijs
DannyvdSluijs force-pushed the fix/correct-phpcs-paths branch from cf420fe to 356cb6a Compare August 21, 2026 08:56
@DannyvdSluijs DannyvdSluijs changed the title Correct php-cs-fixer setup fix: Correct php-cs-fixer setup Aug 21, 2026
@DannyvdSluijs
DannyvdSluijs merged commit f392b29 into main Aug 21, 2026
18 checks passed
@DannyvdSluijs
DannyvdSluijs deleted the fix/correct-phpcs-paths branch August 21, 2026 09:05
github-actions Bot added a commit that referenced this pull request Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.php-cs-fixer.dist.php only check src and tests. The repo contains additional PHP files in other folders.

1 participant