Skip to content

Fix failing deployments by requiring PHP 7 for some tests#567

Closed
danielbachhuber wants to merge 1 commit into
mainfrom
fix/require-php-7-deployment-tests
Closed

Fix failing deployments by requiring PHP 7 for some tests#567
danielbachhuber wants to merge 1 commit into
mainfrom
fix/require-php-7-deployment-tests

Conversation

@danielbachhuber

Copy link
Copy Markdown
Member

No description provided.

@danielbachhuber danielbachhuber added the scope:testing Related to testing label Aug 18, 2023
@danielbachhuber danielbachhuber added this to the 2.9.0 milestone Aug 18, 2023
@danielbachhuber danielbachhuber self-assigned this Aug 18, 2023
@danielbachhuber danielbachhuber requested a review from a team as a code owner August 18, 2023 14:19
@swissspidy

Copy link
Copy Markdown
Member

I think we just need to update the matrix here:

test: #----------------------------------------------------------------------
name: Functional - WP ${{ matrix.wp }} on PHP ${{ matrix.php }} with MySQL ${{ matrix.mysql }}
strategy:
fail-fast: false
matrix:
php: ['5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0']
wp: ['latest']
mysql: ['8.0']
include:

No need to test 5.6 + latest anymore

@danielbachhuber

Copy link
Copy Markdown
Member Author

I think we just need to update the matrix here:

I think you're right #568

@swissspidy swissspidy deleted the fix/require-php-7-deployment-tests branch October 1, 2024 17:35
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.

2 participants