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 ^6.0 for symfony/* #2370

Closed
wants to merge 1 commit into from
Closed

Conversation

malarzm
Copy link
Member

@malarzm malarzm commented Sep 14, 2021

Q A
Type improvement
BC Break no
Fixed issues n/a

Summary

This PR allows installation of upcoming 6.0 version of Symfony's components. Can't really make a test for them as phpbench/phpbench and psalm are in conflict, so let's home the community will let us know if anything is wrong. FWIW I tried running Symfony's ./vendor/bin/patch-type-declarations (https://wouterj.nl/2021/09/symfony-6-native-typing) for 5.4 and we're looking A-OK.

@malarzm malarzm added the Task label Sep 14, 2021
@malarzm malarzm added this to the 2.3.0 milestone Sep 14, 2021
@SenseException
Copy link
Member

The current workflows don't separate between the different Symfony versions. If a build installs a current Symfony 6 and fails, it would break a branch. Should this be addressed in this PR?

@franmomu
Copy link
Contributor

Based on https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5966/files maybe we can do something like franmomu@564263c to test it

@alcaeus
Copy link
Member

alcaeus commented Sep 17, 2021

I have started working on this recently but haven't quite managed to install Symfony 6 yet. For the time being, I decided to remove incompatible development tools (such as phpbench) when running tests, which I think is an acceptable compromise in order to test Symfony 6. For the current WIP state, see https://github.com/alcaeus/mongodb-odm/actions/runs/1236576935.

@IonBazan
Copy link
Member

Closing in favour of #2375.

@IonBazan IonBazan closed this Oct 22, 2021
@malarzm malarzm deleted the sf6-compat branch October 23, 2021 13:52
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.

5 participants