Skip to content

#48 - Update quality stack - #49

Merged
njoubert-cleverage merged 1 commit into
mainfrom
48
Sep 24, 2026
Merged

njoubert-cleverage merged 1 commit into
mainfrom
48

Conversation

@njoubert-cleverage

Copy link
Copy Markdown
Member

Description

Fixes #48

Aligns the demo with the bundles quality stack update (cleverage/process-bundle#191, cleverage/archive-process-bundle#11, cleverage/cache-process-bundle#18, cleverage/doctrine-process-bundle#27, cleverage/flysystem-process-bundle#31, cleverage/rest-process-bundle#29, cleverage/soap-process-bundle#21, cleverage/ui-process-bundle#79).

  • rector.php: withComposerBased(phpunit: true, symfony: true) (PHPUnit was missing)
  • composer.json: declare the Symfony packages used by src/ (symfony/cache, symfony/cache-contracts, symfony/framework-bundle, symfony/http-kernel, symfony/password-hasher, symfony/scheduler) and set a PHPUnit range (^10.5 || ^11 || ^12 || ^13 instead of *)
  • Full quality stack run (rector, php-cs-fixer, phpstan): nothing to fix

Note: unlike the bundles, the demo is a project, so Rector binds version-specific rules to the installed versions (Symfony 7.4, per extra.symfony.require) and not to the lowest constraint. No rule currently changes src/.

Checks (PHP 8.5, Symfony 7.4)

  • rector: no change
  • php-cs-fixer: Fixed 0 of 12 files
  • phpstan: no errors
  • phpunit: no test in the repository yet (No tests executed!)

Requirements

  • Documentation updates
    • Reference
    • Changelog
  • Unit tests

Breaking changes

  • None

🤖 Generated with Claude Code

- rector.php: enable withComposerBased(phpunit: true) alongside symfony
- composer.json: declare Symfony packages used by src/ and set a PHPUnit range
- Run the full quality stack (rector, php-cs-fixer, phpstan): no change needed

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@njoubert-cleverage
njoubert-cleverage merged commit 7cfcd38 into main Sep 24, 2026
17 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.

Update quality stack

1 participant