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

Allow Symfony 7 #89

Merged
merged 11 commits into from
Dec 22, 2023
Merged

Allow Symfony 7 #89

merged 11 commits into from
Dec 22, 2023

Conversation

Lustmored
Copy link
Contributor

This is a simple PR allowing Symfony 7 components to be used with this bundle.

@peter-gribanov
Copy link
Member

Tests fail. Please see what the problem might be.

@Lustmored
Copy link
Contributor Author

Tests fail. Please see what the problem might be.

Tests are failing on code coverage upload. I'm afraid it might be out of my reach to fix that, possibly something on the other side? It fails only for one case where I've changed PHP and Sf version to supported one (Sf 6.0->6.4 and PHP 8.0->8.1 per Sf requirements and new PHP 8.2 + Sf 7).

But I'll fix the CS for sure (errors are out of my changes, but that's trivial to fix anyways, so no problem).

@Lustmored
Copy link
Contributor Author

@peter-gribanov I think I have fixed both code style and CI workflow so that it can pass, but the checks needs approval to be sure 👍

@Lustmored
Copy link
Contributor Author

I have ran them elsewhere only to find phpstan having some troubles. Sorry for the noise, I will investigate further...

@Lustmored Lustmored force-pushed the symfony-7 branch 3 times, most recently from f0b54c0 to ff3acae Compare December 21, 2023 12:31
@Lustmored
Copy link
Contributor Author

@peter-gribanov I think I have won with phpstan. I cannot test code coverage upload, so please accept actions to see. I had to bump both phpstan and PHP minimal version (the latter is required by phpstan) and adapt to changes within.

How do you feel about dropping support for unmaintained PHP and/or Symfony versions? I can step up and create a PR with such work if that sounds good to you (or maybe even go for it in this one?)

@peter-gribanov
Copy link
Member

peter-gribanov commented Dec 22, 2023

Good job. Thank you 👍

@peter-gribanov peter-gribanov merged commit ae2a9e8 into gpslab:master Dec 22, 2023
19 checks passed
@peter-gribanov
Copy link
Member

I made a new release Releases 2.1.0

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