Skip to content

Bump lowest PHP version to test to 5.4#4798

Closed
schlessera wants to merge 1 commit intomasterfrom
skip-testing-on-php-53
Closed

Bump lowest PHP version to test to 5.4#4798
schlessera wants to merge 1 commit intomasterfrom
skip-testing-on-php-53

Conversation

@schlessera
Copy link
Copy Markdown
Member

The build for testing WP-CLI is currently broken on PHP 5.3. Given that:

  • the current version v2.x is supposed to drop PHP 5.3 support;
  • PHP 5.3 only runs on Travis within a separate distro, which might not get the same care;
  • efforts in Travis builds fail with a missing scaffold package command #4791 to fix the PHP 5.3 build have failed;
  • no bug reports were provided for the issue the tests on PHP 5.3 flag;

=> I've decided to just bump the minimum tested version to PHP 5.4 now.

Fixes #4791

The build for testing WP-CLI is currently broken on PHP 5.3. Given that:

- the current version v2.x is supposed to drop PHP 5.3 support;
- PHP 5.3 only runs on Travis within a separate distro, which might not get the same care;
- efforts in #4791 to fix the PHP 5.3 build have failed;
- no bug reports were provided for the issue the tests on PHP 5.3 flag;

=> I've decided to just bump the minimum tested version to PHP 5.4 now.

Fixes #4791
@schlessera schlessera added the scope:testing Related to testing label May 15, 2018
@schlessera schlessera added this to the 2.0.0 milestone May 15, 2018
@schlessera schlessera requested a review from a team May 15, 2018 15:57
@danielbachhuber
Copy link
Copy Markdown
Member

Build failure here too?

@schlessera
Copy link
Copy Markdown
Member Author

Yes, which renders my theory of the problem being specific to the 5.3 Travis environment invalid.

@schlessera
Copy link
Copy Markdown
Member Author

Test failure was fixed via wp-cli/package-command#83

Closing this for now as it is not needed yet.

@schlessera schlessera closed this May 29, 2018
@schlessera schlessera deleted the skip-testing-on-php-53 branch May 29, 2018 12:54
@danielbachhuber
Copy link
Copy Markdown
Member

Test failure was fixed via wp-cli/package-command#83

Nice!

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

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Travis builds fail with a missing scaffold package command

2 participants