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

Update PHP-CS-Fixer to support PHP 7.4 #141

Merged
merged 3 commits into from
Feb 4, 2020
Merged

Conversation

X-Coder264
Copy link
Collaborator

No description provided.

@X-Coder264
Copy link
Collaborator Author

#137 was not enough. Even though we no longer depend on symfony/contracts, symfony/symfony 4.2 still does so we are running into the same problem. Since dropping SF 4.2 in the CI is IMO not an option, we will have to wait on PHP-CS-Fixer/PHP-CS-Fixer#4685 to be merged and tagged before we can proceed with this PR.

@rbaarsma
Copy link

@X-Coder264 I've noticed this issue is already open for 2 weeks, because the related issue is still open. And my PR #118 is also waiting on this.

I'm wondering why you don't want to drop symfony 4.2 support? It's no longer maintained. As I've done in my PR, the minimum symfony version for newer versions of this bundle should, in my opinion, be 4.4. Not sure if that fixes the issue?

@X-Coder264
Copy link
Collaborator Author

@rbaarsma I've pinged the maintainers in the PHP-CS fixer repo to see what is the status on merging the fix. Hopefully it's gonna be merged and tagged soon. If not we will explore other options.

As far as your question regarding Symfony 4.2 - we don't like dropping support for Symfony versions in minor releases (unless there is a significant maintenance burden which isn't the case with SF 4.2). A lot of people are still using 4.2 for whatever reasons and from our POV there is no need to force them to upgrade if they want to keep getting the bug fixes and new stuff that gets merged in the 2.x branch.

@rbaarsma
Copy link

rbaarsma commented Jan 9, 2020

@X-Coder264 seems the story where this PR was waiting for is merged. Perhaps this can be merged too and then the php 7.4 and symfony 5 story as well?

@rbaarsma
Copy link

rbaarsma commented Jan 9, 2020

Let me know where I can help

@X-Coder264
Copy link
Collaborator Author

@rbaarsma Yes, the CS-fixer PR got merged, but they haven't tagged a new version with the fix yet. As soon as they do that (hopefully in the next couple of days) we will merge this and let you know so that you can finish your PR.

@spideyfusion spideyfusion force-pushed the update-php-cs-fixer branch 3 times, most recently from ce1650b to a1cdc5b Compare February 3, 2020 21:11
@X-Coder264 X-Coder264 merged commit ecb1feb into v2.x Feb 4, 2020
@X-Coder264 X-Coder264 deleted the update-php-cs-fixer branch February 4, 2020 10:50
@spideyfusion spideyfusion added this to the v2.1.1 milestone Feb 26, 2020
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