Skip to content

Bump the minimim version of php to 8.0#727

Merged
ingeniumed merged 1 commit intodevelopfrom
update/php
May 28, 2024
Merged

Bump the minimim version of php to 8.0#727
ingeniumed merged 1 commit intodevelopfrom
update/php

Conversation

@ingeniumed
Copy link
Copy Markdown
Contributor

Description

Bump the minimum version of php to 8.0, as well as the minimum WP version to 6.0. As a result of this, the packages in composer.json have been updated and the GH workflow has been modified to run against 8.0/6.0 rather than 7.4/5.9.

@ingeniumed ingeniumed requested a review from alecgeatches May 28, 2024 09:36
@ingeniumed ingeniumed self-assigned this May 28, 2024
Copy link
Copy Markdown
Contributor

@alecgeatches alecgeatches left a comment

Choose a reason for hiding this comment

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

Looks good! We can address the failing e2e tests (unrelated) in #726.

@ingeniumed ingeniumed merged commit 0bd10bd into develop May 28, 2024
@ingeniumed ingeniumed deleted the update/php branch May 28, 2024 09:40
Comment thread edit_flow.php
Comment on lines +10 to +11
* License: GPL-3
* License URI: https://www.gnu.org/licenses/gpl-3.0.html
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Was the license change intentional?

composer.json still says GPL v2 or later. What is it?

I'd recommend adding a proper LICENSE file to the repo.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching that. We meant to make it GPL 3 as some of our other plugins have that already, so we were trying to make it consistent. I'll fix that inconsistency, and add a proper LICENSE file to the repo.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This was an incorrect decision. A site that uses a GPLv2-only plugin can't legally be allowed to work with a version of Edit-Flow that is GPLv3 only. I've opened #804 to rectify this.

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.

4 participants