Run CI with various versions of Symfony #518
Conversation
GromNaN
force-pushed
the
ci-symfony-versions
branch
3 times, most recently
from
177ea30 to
78ee9ed
Compare
GromNaN
force-pushed
the
ci-symfony-versions
branch
from
78ee9ed to
f9e73bc
Compare
| symfony: '7.3.*' | ||
| - php: '8.3' | ||
| monolog: '3.*' | ||
| symfony: '7.3.*' |
There was a problem hiding this comment.
This is the wrong approach to solve this problem IMO, as it means that we have to remember to update the CI of monolog-bundle every 6 months when Symfony releases new versions, otherwise we don't test latest versions (while our requirements allow them).
A cleaner solution is to have most job run with original composer requirements (so no SYMFONY_REQUIRE), and to add extra jobs forcing a Symfony version for the supported LTS versions (which need maintenance only every 2 years when LTS versions change).
Non-LTS versions will be installed by the normal jobs as Composer installs the latest compatible version
See https://github.com/FriendsOfSymfony/FOSUserBundle/blob/master/.github/workflows/ci.yaml for an example setup (where I'm not skipping the definition of the env variable when matrix.symfony is false but I'm skipping the installation of Flex, which consumes it, producing an equivalent result)
Ensure the bundle is compatible with all the supported Symfony versions.