Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix CI #2162

Merged
merged 5 commits into from
Dec 31, 2023
Merged

fix CI #2162

merged 5 commits into from
Dec 31, 2023

Conversation

smoench
Copy link
Contributor

@smoench smoench commented Dec 5, 2023

This PR adjusted following:

  • uses ubuntu-22.04 as latest runner image
  • uses actions/checkout@v4 to prevent deprecations
  • uses actions/cache@v3 to prevent deprecations
  • replaces deprecated set-ouput with $GITHUB_OUTPUT
  • adds flex via shivammathur/setup-php tools
  • uses direct dev dependency for phpunit as it faster, cached by actions composer cache and fixes an error on PHP 7.2
  • adds Symfony 6.4 to the test matrix and updates deprecation baseline

@DjordyKoert
Copy link
Collaborator

Nice!

- php-version: 8.1
symfony-require: "6.3.*"
- php-version: 8.2
symfony-require: "6.4.*"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Php 8.1 is still supported by this version, we could also add a test for this

@DjordyKoert
Copy link
Collaborator

@GuilhemN ci fails because a deprecation warning is not present in the phpunit baseline. Generating a new baseline should fix it.

1x: The "Sensio\Bundle\FrameworkExtraBundle\Routing\AnnotatedRouteControllerLoader" class is deprecated since version 5.2. Use "Symfony\Bundle\FrameworkBundle\Routing\AnnotatedRouteControllerLoader" instead.

@GuilhemN GuilhemN merged commit 4c67756 into nelmio:master Dec 31, 2023
7 of 8 checks passed
@GuilhemN
Copy link
Collaborator

Thanks a lot @smoench! Best wishes for the new year.

@DjordyKoert
Copy link
Collaborator

DjordyKoert commented Dec 31, 2023

Also small question from my end @GuilhemN. Are you planning to make a new release for the Symfony 7 support after this is merged?

@smoench smoench deleted the fix-ci branch December 31, 2023 17:55
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.

3 participants