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

Handle files with the .php8 extension #263

Conversation

@alexholt
Copy link
Contributor

@alexholt alexholt commented Feb 11, 2021

This is necessary in order for the phar to include:
symfony/polyfill-mbstring/Resources/mb_convert_variables.php8

This is necessary in order for the phar to include
symfony/polyfill-mbstring/Resources/mb_convert_variables.php8
@alexholt alexholt requested a review from as a code owner Feb 11, 2021
Copy link
Member

@swissspidy swissspidy left a comment

Interesting!

@swissspidy
Copy link
Member

@swissspidy swissspidy commented Feb 12, 2021

I think we could just bump the composer dependency instead.

@swissspidy swissspidy dismissed their stale review Feb 12, 2021

changed my mind

But they are not needed for this
@alexholt alexholt force-pushed the handle-php8-file-extension branch from 5bcffb4 to 13c61fc Feb 13, 2021
@@ -143,11 +143,11 @@ jobs:
- name: Prepare test database
if: steps.check_files.outputs.files_exists == 'true'
run: |
export MYQSL_HOST=127.0.0.1
export MYSQL_HOST=127.0.0.1
Copy link
Member

@schlessera schlessera Feb 15, 2021

🤦

@schlessera schlessera changed the base branch from master to fix/gha-functional-tests-update Mar 5, 2021
@schlessera schlessera added this to the 2.5.0 milestone Mar 5, 2021
@schlessera schlessera added the bug label Mar 5, 2021
@schlessera schlessera merged commit 3cbbf02 into wp-cli:fix/gha-functional-tests-update Mar 5, 2021
20 of 24 checks passed
20 of 24 checks passed
@github-actions[bot]
Lint PHP files
Details
@github-actions[bot]
Unit test / PHP 5.6
Details
@github-actions[bot]
Unit test / PHP 7.0
Details
@github-actions[bot]
Unit test / PHP 7.1
Details
@github-actions[bot]
Unit test / PHP 7.2
Details
@github-actions[bot]
Unit test / PHP 7.3
Details
@github-actions[bot]
Unit test / PHP 7.4
Details
@github-actions[bot]
Unit test / PHP 8.0
Details
@github-actions[bot]
PHPCS
Details
@github-actions[bot]
Functional - WP latest on PHP 5.6 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 7.0 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 7.1 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 7.2 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 7.3 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 7.4 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP latest on PHP 8.0 with MySQL 8.0 Functional - WP latest on PHP 8.0 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP trunk on PHP 5.6 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP trunk on PHP 5.6 with MySQL 5.7
Details
@github-actions[bot]
Functional - WP trunk on PHP 5.6 with MySQL 5.6
Details
@github-actions[bot]
Functional - WP trunk on PHP 7.4 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP trunk on PHP 8.0 with MySQL 8.0 Functional - WP trunk on PHP 8.0 with MySQL 8.0
Details
@github-actions[bot]
Functional - WP trunk on PHP 8.0 with MySQL 5.7 Functional - WP trunk on PHP 8.0 with MySQL 5.7
Details
@github-actions[bot]
Functional - WP trunk on PHP 8.0 with MySQL 5.6 Functional - WP trunk on PHP 8.0 with MySQL 5.6
Details
@github-actions[bot]
Functional - WP 3.7 on PHP 5.6 with MySQL 5.6
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants