Skip to content

Conversation

@Asymtode712
Copy link

@Asymtode712 Asymtode712 commented Oct 1, 2025

PR Summary

Successfully updated minimum requirements to PHP 8.3+ and Symfony 6.4+

Changes Made

1. composer.json Dependencies Updated

  • PHP: ^8.1^8.3
  • All Symfony components: ^5.4 || ^6.0 || ^7.0^6.4 || ^7.0
  • jms/metadata: ^2.4^2.8
  • symfony/event-dispatcher-contracts: ^3.1^3.5
  • doctrine/doctrine-bundle: ^2.7^2.13
  • doctrine/mongodb-odm: ^2.4^2.8
  • doctrine/orm: ^2.13 || ^3.0^2.19 || ^3.0
  • matthiasnoback/symfony-dependency-injection-test: ^5.1^6.0

2. Docker Configuration

  • Removed Dockerfile81 (PHP 8.1)
  • Removed Dockerfile82 (PHP 8.2)
  • Updated Makefile default TARGET from 81 to 83

3. CI/CD Configuration (.github/workflows/build.yaml)

  • Removed PHP 8.1 and 8.2 from test matrix
  • Removed Symfony 5.4 from test matrix
  • Now tests with PHP 8.3 and 8.4 only
  • Tests Symfony 6.4 and 7.x only

4. Documentation Updates

  • CONTRIBUTING.md: Updated to reflect PHP 8.3+ as default environment
  • docs/symfony_support_policy.md: Ready for update (mentions versions)

5. Test Fixes

  • tests/Naming/SlugNamerTest.php: Fixed PHPUnit 10-12 compatibility by using anonymous class instead of deprecated addMethods() API
  • tests/VichUploaderBundleTest.php: Updated PHP version check from <= 80200 to < 80300

Test Results

✅ PHP 8.3 Tests - PASSED

  • Total Tests: 308
  • Assertions: 1,215
  • Passed: 308
  • Errors: 0
  • Failures: 0
  • Skipped: 14 (intentional)
  • Risky: 15 (warnings about exception handlers, not failures)
  • Status: ✅ ALL TESTS PASSING

fixes #1544

- Update minimum PHP version to ^8.3
- Update all Symfony components to ^6.4 || ^7.0
- Upgrade other dependencies to compatible versions
- Remove Docker files for PHP 8.1 and 8.2
- Update CI/CD to test only with PHP 8.3+
- Fix test compatibility with PHPUnit 10-12
- Update documentation to reflect new requirements
@Asymtode712
Copy link
Author

@garak Would you please review this PR

@garak
Copy link
Collaborator

garak commented Oct 2, 2025

Please edit the PR to point to the 3.x branch

@Asymtode712 Asymtode712 closed this Oct 2, 2025
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.

Drop old dependencies

2 participants