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

Drop support for Symfony < 5.4, add Symfony 6.4 and PHP 8.3 to CI #342

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ker0x
Copy link
Contributor

@ker0x ker0x commented Sep 25, 2023

This PR drop support for Symfony < 5.4 and add 6.3 to the CI test matrix. It also update the .gitattributes that was outdated.

@norkunas
Copy link
Member

norkunas commented Oct 2, 2023

Thanks. Before merging this we should wait for geocoder-php/Geocoder#1184

@ker0x ker0x changed the title Drop support for Symfony < 5.4, add Symfony 6.3 to CI, update .gitattributes file Drop support for Symfony < 5.4, add Symfony 6.4 and PHP 8.3 to CI, update .gitattributes file Dec 16, 2023
@ker0x
Copy link
Contributor Author

ker0x commented Dec 16, 2023

Friendly ping @norkunas! I have rebased my PR and include Symfony 6.4 and PHP 8.3 to the CI matrix !

@ker0x ker0x force-pushed the feature/drop-sf-4.4 branch 2 times, most recently from 9dd2c51 to 5cdcc2f Compare December 16, 2023 19:00
@norkunas
Copy link
Member

norkunas commented Dec 16, 2023

Sorry,will be available on wednesday,and will review 🙌

@ker0x ker0x force-pushed the feature/drop-sf-4.4 branch 13 times, most recently from 70f28d1 to 5c911ef Compare January 29, 2024 11:28
@ker0x
Copy link
Contributor Author

ker0x commented Jan 29, 2024

@norkunas tests are green on my side (https://github.com/ker0x/geocoder-php-geocoder-bundle/actions/runs/7695337920). I will prefer to see this PR merge instead of #350 as there are too many problems with deprecations and Symfony 4.4

@norkunas
Copy link
Member

I prefer to make a last release with fixed tests and then dropping support for 4.4

@ker0x ker0x changed the title Drop support for Symfony < 5.4, add Symfony 6.4 and PHP 8.3 to CI, update .gitattributes file Drop support for Symfony < 5.4, add Symfony 6.4 and PHP 8.3 to CI Jan 29, 2024
@ker0x ker0x force-pushed the feature/drop-sf-4.4 branch 6 times, most recently from bf60223 to d226cf4 Compare January 30, 2024 09:50
@@ -101,7 +91,6 @@ jobs:
uses: "ramsey/composer-install@v2"

Choose a reason for hiding this comment

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

Version 3 was released this week. This could be changed to fix node16 => 20 deprecation in ci

@norkunas
Copy link
Member

@ker0x as `geocoder-php/Geocoder release is stalled, I think we can finish/merge this and release which will save us some trouble, because it's too hard to maintain such a broad version range so I had to completely disable 2 CI jobs

Comment on lines 71 to 73
- php: '7.4'
sf_version: '5.4.*'
dependencies: 'lowest'
Copy link
Member

Choose a reason for hiding this comment

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

we should probably leave this, only removing 4.4

Copy link
Member

Choose a reason for hiding this comment

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

WDYT?

@ker0x ker0x force-pushed the feature/drop-sf-4.4 branch 3 times, most recently from f3c3c79 to 7ee9558 Compare May 28, 2024 06:27
composer.json Outdated Show resolved Hide resolved
tests/allowed.json Outdated Show resolved Hide resolved
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