Skip to content

Conversation

@derrabus
Copy link
Member

@derrabus derrabus commented Nov 29, 2025

  • We can support Symfony Console 8 in ORM 2, so let's do that.
  • We cannot use Symfony Cache 8 in our tests because it requires DBAL 4 which we don't support on ORM 2.
  • We don't need to test against Symfony VarExporter 8 because all the features we use from that package have been removed in 8.0.

@derrabus derrabus added this to the 2.20.9 milestone Nov 29, 2025
@derrabus derrabus added the Bug label Nov 29, 2025
@derrabus derrabus requested a review from greg0ire November 29, 2025 13:03
Copy link
Member

@greg0ire greg0ire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, but you know what I think about treating this like a bugfix. In addition to this, merging this up to 3.5 when you contributed sf8 support to 3.6 makes little sense. Still, I'm approving because I don't want the sf hackathon momentum to get lost.

@derrabus derrabus merged commit 87f1ba7 into doctrine:2.20.x Nov 29, 2025
92 checks passed
@derrabus derrabus deleted the improvement/symfony-8 branch November 29, 2025 14:04
@derrabus
Copy link
Member Author

I'll merge it up to 3.5, but the whole trait will be gone there.

derrabus added a commit to derrabus/orm that referenced this pull request Nov 29, 2025
* 2.20.x:
  Support Symfony Console 8 (doctrine#12300)
  Bump doctrine/.github/.github/workflows/composer-lint.yml (doctrine#12288)
  Bump doctrine/.github/.github/workflows/documentation.yml (doctrine#12289)
  Bump doctrine/.github/.github/workflows/coding-standards.yml (doctrine#12290)
  Bump doctrine/.github/.github/workflows/release-on-milestone-closed.yml (doctrine#12291)
  Bump actions/checkout from 5 to 6 (doctrine#12292)
derrabus added a commit to derrabus/orm that referenced this pull request Nov 29, 2025
* 3.5.x:
  Removes Guides from our dependencies (doctrine#12303)
  Fix PHPStan and test errors after DBAL 4.4 and Symfony 7.4 releases (doctrine#12301)
  Support Symfony Console 8 (doctrine#12300)
  Bump doctrine/.github/.github/workflows/composer-lint.yml (doctrine#12288)
  Bump doctrine/.github/.github/workflows/documentation.yml (doctrine#12289)
  Bump doctrine/.github/.github/workflows/coding-standards.yml (doctrine#12290)
  Bump doctrine/.github/.github/workflows/release-on-milestone-closed.yml (doctrine#12291)
  Bump actions/checkout from 5 to 6 (doctrine#12292)
derrabus added a commit that referenced this pull request Nov 29, 2025
* 3.5.x:
  Explicitly set a cache in testDisablingXmlValidationIsPossible (#12307)
  Removes Guides from our dependencies (#12303)
  Fix PHPStan and test errors after DBAL 4.4 and Symfony 7.4 releases (#12301)
  Support Symfony Console 8 (#12300)
  Bump doctrine/.github/.github/workflows/composer-lint.yml (#12288)
  Bump doctrine/.github/.github/workflows/documentation.yml (#12289)
  Bump doctrine/.github/.github/workflows/coding-standards.yml (#12290)
  Bump doctrine/.github/.github/workflows/release-on-milestone-closed.yml (#12291)
  Bump actions/checkout from 5 to 6 (#12292)
derrabus added a commit that referenced this pull request Nov 29, 2025
* 3.5.x:
  Remove obsolete VarExporter feature detection (#12309)
  Allow Symfony 8 (#12308)
  Explicitly set a cache in testDisablingXmlValidationIsPossible (#12307)
  Removes Guides from our dependencies (#12303)
  Fix PHPStan and test errors after DBAL 4.4 and Symfony 7.4 releases (#12301)
  Support Symfony Console 8 (#12300)
  Bump doctrine/.github/.github/workflows/composer-lint.yml (#12288)
  Bump doctrine/.github/.github/workflows/documentation.yml (#12289)
  Bump doctrine/.github/.github/workflows/coding-standards.yml (#12290)
  Bump doctrine/.github/.github/workflows/release-on-milestone-closed.yml (#12291)
  Bump actions/checkout from 5 to 6 (#12292)
derrabus added a commit to derrabus/orm that referenced this pull request Nov 30, 2025
* 3.6.x:
  Remove obsolete PHPStan ignore rules
  Remove obsolete VarExporter feature detection (doctrine#12309)
  Allow Symfony 8 (doctrine#12308)
  Explicitly set a cache in testDisablingXmlValidationIsPossible (doctrine#12307)
  Removes Guides from our dependencies (doctrine#12303)
  Fix PHPStan and test errors after DBAL 4.4 and Symfony 7.4 releases (doctrine#12301)
  Support Symfony Console 8 (doctrine#12300)
  Bump doctrine/.github/.github/workflows/composer-lint.yml (doctrine#12288)
  Bump doctrine/.github/.github/workflows/documentation.yml (doctrine#12289)
  Bump doctrine/.github/.github/workflows/coding-standards.yml (doctrine#12290)
  Bump doctrine/.github/.github/workflows/release-on-milestone-closed.yml (doctrine#12291)
  Bump actions/checkout from 5 to 6 (doctrine#12292)
  Add ORDER BY clause to more test cases
  Fix check for composite foreign key
  Fix documentation about default values
  Fix eager fetch composite foreign key (doctrine#11397)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants