Skip to content

Dev version 4#85

Draft
Spamercz wants to merge 12 commits into
masterfrom
dev-4
Draft

Dev version 4#85
Spamercz wants to merge 12 commits into
masterfrom
dev-4

Conversation

@Spamercz
Copy link
Copy Markdown
Member

This pull request updates dependencies to support newer versions of PHP and related libraries, refactors code to use modern Symfony console helpers, improves type safety in test mocks, and makes configuration changes for Composer and PHPStan. These changes help keep the codebase current, improve reliability, and ensure compatibility with the latest tooling.

Dependency and compatibility updates:

  • Updated the minimum required PHP version to ^8.2 and bumped versions for several dependencies in composer.json, including nette/utils, php-amqplib/php-amqplib, latte/latte, tracy/tracy, and phpstan/phpstan. Added symfony/console as a new dev dependency.
  • Updated the Composer config to add plugin allowances and removed the branch-alias setting.

Code modernization and refactoring:

  • Refactored the confirmation prompt in PurgeConsumerCommand.php to use Symfony's ConfirmationQuestion instead of the deprecated dialog helper, improving code clarity and future compatibility.
  • Improved type safety in ChannelMock.php by adding explicit type hints to method parameters and return types, and updated parent method calls accordingly.

Configuration and tooling:

  • Updated PHPStan configuration by replacing excludes_analyse with excludePaths for better compatibility with newer PHPStan versions.
  • Cleaned up the test configuration in nette-reset.neon by removing unused or deprecated service definitions.

- drop kdyby console specific dependency and replaced with core symfony dependency (for dev) you are free to choose console integration on your project (hopefully :) )
@Spamercz Spamercz self-assigned this Sep 17, 2025
- drop kdyby console specific dependency and replaced with core symfony dependency (for dev) you are free to choose console integration on your project (hopefully :) )
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.

1 participant