Resolve the vendor/wordpress-autoload.php file on Windows, add PHP 8.1-8.2 support #29
Conversation
srtfisher
changed the title
|
|
||
| jobs: | ||
| test: | ||
| runs-on: ${{ matrix.os }} | ||
| strategy: | ||
| fail-fast: true | ||
| matrix: | ||
| os: [ubuntu-latest] | ||
| php: [7.4, 8.0, 8.1] | ||
| os: [ubuntu-latest, windows-2019] |
| on: | ||
| push: | ||
| branches: | ||
| - main |
There was a problem hiding this comment.
This will also trigger after the PR is merged. Is that intentional?
There was a problem hiding this comment.
Yes, I prefer to always have a status check on the main branch to get a valid status badge.
| preg_replace('#/vendor/.*$#', '/vendor/wordpress-autoload.php', __DIR__), | ||
| preg_replace('/\\\vendor\\\.*$/', '/vendor/wordpress-autoload.php', __DIR__), |
There was a problem hiding this comment.
This looks like one too many backslashes - shouldn't it be like this?
| preg_replace('/\\\vendor\\\.*$/', '/vendor/wordpress-autoload.php', __DIR__), | |
| preg_replace('/\\vendor\\.*$/', '/vendor/wordpress-autoload.php', __DIR__), |
There was a problem hiding this comment.
I'm not a regex pro to know but in testing with C:\Users\Sean Fisher\Local Sites\wordpress\app\public\wp-content\vendor\alleyinteractive\composer-wordpress-autoloader\src as the __DIR__:
four slashes: preg_replace('/\\\vendor\\\.*$/', '/vendor/wordpress-autoload.php', __DIR__), -> "C:\Users\Sean Fisher\Local Sites\wordpress\app\public\wp-content/vendor/wordpress-autoload.php
two slashes: preg_replace('/\\vendor\\\.*$/', '/vendor/wordpress-autoload.php', __DIR__), -> C:\Users\Sean Fisher\Local Sites\wordpress\app\public\wp-content\vendor\alleyinteractive\composer-wordpress-autoloader\src
/shrug
Changed
Fixed
wordpress-autoload.phpfile not properly loading on Windows.Fixes #28