Skip to content

Document PHP compatibility #15803

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

Merged
merged 1 commit into from
Sep 14, 2021
Merged

Conversation

lol768
Copy link
Contributor

@lol768 lol768 commented Sep 12, 2021

ref symfony/symfony#42993

This makes it, at least to me, a lot clearer what the policy is re which Symfony versions will support which versions of PHP. Please correct me if I've misunderstood anything - I am most unsure about how the minimal PHP version is chosen - obviously I've seen the RFC labeled issues, but am I correct in assuming this is ultimately decided by the core team before each major release?

I suggested mentioning this on https://symfony.com/releases too but I think the website might be closed-source?

@carsonbot carsonbot added this to the 4.4 milestone Sep 12, 2021
Comment on lines +107 to +108
The **minimum** PHP version is decided for each major Symfony version by consensus
amongst the :doc:`core team </contributing/code/core_team>` and documented as
Copy link
Contributor

@ro0NL ro0NL Sep 13, 2021

Choose a reason for hiding this comment

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

i was also curious if there's some rule maybe? IIUC new SF major requires latest PHP major at time of release, right? in that sense, is it predictable?

Copy link
Member

@derrabus derrabus Sep 13, 2021

Choose a reason for hiding this comment

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

I wouldn't dare to predict the min PHP version for Symfony 7, 8 and 9 years in advance, if that's what you mean. 😉

Related: symfony/symfony#40389

Copy link
Contributor

@ro0NL ro0NL Sep 13, 2021

Choose a reason for hiding this comment

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

then i predict SF 8 will require PHP 9 ;)

Copy link
Member

Choose a reason for hiding this comment

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

I'm gonna bookmark this discussion. Let's see how well it ages. 😁

Copy link
Member

@wouterj wouterj left a comment

Choose a reason for hiding this comment

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

Thank you. I think this is a useful addition!

am I correct in assuming this is ultimately decided by the core team before each major release?

Yes, you are :)

@javiereguiluz
Copy link
Member

Adam, thanks for this contribution ... and congrats on your first Symfony Docs contribution 🎉

@javiereguiluz javiereguiluz merged commit 798bbfc into symfony:4.4 Sep 14, 2021
@lol768
Copy link
Contributor Author

lol768 commented Sep 15, 2021

Thanks all for reviewing :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants