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 PHP 7.0 and PHP 7.1 #29

Merged
merged 1 commit into from
Oct 13, 2020
Merged

Drop PHP 7.0 and PHP 7.1 #29

merged 1 commit into from
Oct 13, 2020

Conversation

micgro42
Copy link
Collaborator

These versions are no longer supported by MediaWiki, Wikibase or PHP itself. Also, they are blocking #28.

This currently keeps PHP 7.2 as Wikimedia production is still on PHP 7.2 and will need a bit more time to change: https://phabricator.wikimedia.org/T257879#6339607

Depending on how one wants to look at it, this might be seen as a breaking change that requires a new major version release? That would seem to be related to the discussion in #27, the result of which isn't entirely clear to me.

These versions are no longer supported by MediaWiki, Wikibase or PHP
itself. Also, they are blocking #28.

This currently keeps PHP 7.2 as Wikimedia production is still on PHP 7.2
and will need a bit more time to change: https://phabricator.wikimedia.org/T257879#6339607
Copy link
Member

@lucaswerkmeister lucaswerkmeister left a comment

Choose a reason for hiding this comment

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

I don’t think dropping support for already unsupported PHP versions should require a major version bump – that seems impractical, even if I can see the reasoning behind it.

@thiemowmde
Copy link
Contributor

I agree. I can't explain it well, but from what I remember it was often argued that such changes should not qualify as breaking ones.

@itamargiv itamargiv merged commit 19df59d into master Oct 13, 2020
@itamargiv itamargiv deleted the dropOldPHP branch October 13, 2020 09:50
@JeroenDeDauw
Copy link
Contributor

There is no need to bump the major version for requirement changes to PHP or dependent upon packages since composer already takes care of that.

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

Successfully merging this pull request may close these issues.

5 participants