| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…WordPress compatibility
… PHPUnit compatibility
| - php: "8.4" | ||
| wp: "latest" | ||
| experimental: false | ||
| # Lowest supported: PHP 7.4 + oldest WP. |
There was a problem hiding this comment.
I am not completely sure why this was made. The WordPress importer, like all the other standard ones, supports down to WP 5.4 and WordPress down to PHP 7.2. To fix GHSA-vvj3-c3rp-c85p unit tests failing? If that's the only reason, it's ok.
But please make these two changes on https://github.com/WordPress/substack-importer/blob/trunk/readme.txt:
A new version for these only changes is not needed. At the next one please add this line:
= x.y.z =
1. Compatibility: the plugin now requires PHP 7.4 or higher.
Sorry, something went wrong.
There was a problem hiding this comment.
Yes, it was to fix the unit tests failing in GHSA-vvj3-c3rp-c85p. PHPUnit ^9.6 is now used, which has the issue fixed. I can probably add continue-on-error: true for older versions and use a lower PHPUnit, but I figured that it's okay like this. Let me know if I should do it that way, and I'll fix it in another PR.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Changes proposed in this Pull Request
Testing instructions