Skip to content

Fix compatibility with symfony/console 8.x#1503

Merged
stof merged 1 commit intophpspec:mainfrom
mpdude:patch-1
Jan 20, 2026
Merged

Fix compatibility with symfony/console 8.x#1503
stof merged 1 commit intophpspec:mainfrom
mpdude:patch-1

Conversation

@mpdude
Copy link
Copy Markdown
Contributor

@mpdude mpdude commented Dec 17, 2025

The add() method has been removed in Symfony 8.x.

addCommands() has been available since long ago and can be used to be cross-version compatible.

The `add()` method has been removed in Symfony 8.x.

`addCommands()` has been available since long ago and can be used to be cross-version compatible.
Rafikooo added a commit to Rafikooo/SyliusMailerBundle that referenced this pull request Jan 13, 2026
phpspec 8.1 has a compatibility issue with symfony/console 8.x
that causes 'Call to undefined method Application::add()' error.
PHPUnit tests still run for Symfony 8 to validate compatibility.
roukmoute added a commit to roukmoute/polyfill-calendar that referenced this pull request Jan 16, 2026
- Require PHP 8.0+
- Update CI test matrix to PHP 8.0-8.3
- Upgrade to php-cs-fixer v3
- Update dev dependencies (phpstan ^1.0, psalm ^5.0)
- Apply PHP 8 code style (nullable types syntax)
- Remove obsolete PedroTroller/single_line_comment fixer
- Bump version to 2.0-dev

Note: PHP 8.4 tests skipped due to phpspec bug with Symfony Console 8
(see phpspec/phpspec#1503)
roukmoute added a commit to roukmoute/polyfill-calendar that referenced this pull request Jan 16, 2026
- Require PHP 8.0+
- Update CI test matrix to PHP 8.0-8.3
- Upgrade to php-cs-fixer v3
- Update dev dependencies (phpstan ^1.0, psalm ^5.0)
- Apply PHP 8 code style (nullable types syntax)
- Remove obsolete PedroTroller/single_line_comment fixer
- Bump version to 2.0-dev

Note: PHP 8.4 tests skipped due to phpspec bug with Symfony Console 8
(see phpspec/phpspec#1503)
@stof stof merged commit 3ea90cc into phpspec:main Jan 20, 2026
12 checks passed
@mpdude mpdude deleted the patch-1 branch January 20, 2026 09:29
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