Skip to content

Conversation

@W0rma
Copy link

@W0rma W0rma commented Dec 12, 2025

This PR fixes some deprecation warnings which I noticed when running the tests locally.

Furthermore, it starts using the plain array symfony config syntax because config builders were deprecated in symfony 7.4 (https://symfony.com/blog/new-in-symfony-7-4-better-php-configuration).

@W0rma
Copy link
Author

W0rma commented Dec 18, 2025

@TavoNiievez Do you want to create a new 7.4 branch or rename the existing 7.3 branch? In both cases the branch name needs to be adjusted in .github/workflows/symfony.yml and in the CI config of https://github.com/Codeception/module-symfony accordingly.

@W0rma
Copy link
Author

W0rma commented Dec 18, 2025

Btw I already played with symfony 8 locally and could submit another PR as soon as this one got accepted.

@TavoNiievez Please let me know if you want to create the 8.0 branch beforehand.

@TavoNiievez TavoNiievez changed the base branch from 7.3 to 7.4 December 23, 2025 22:37
@TavoNiievez TavoNiievez merged commit e497462 into Codeception:7.4 Dec 23, 2025
3 checks passed
@TavoNiievez
Copy link
Member

TavoNiievez commented Dec 23, 2025

@W0rma Please update the branch name in the code of the fork in this file before opening the PR next time:
https://github.com/Codeception/symfony-module-tests/blob/e497462c3a33342dd46a48fc9c97d89b4b500feb/.github/workflows/symfony.yml#L6C10-L6C13
I will check the CI execution during the push.
The branch creation can happen afterwards.

@W0rma W0rma deleted the 7.4 branch December 24, 2025 06:32
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