Provide PHP 8.1 support

laminas / laminas-escaper

Securely and safely escape HTML, HTML attributes, JavaScript, CSS, and URLs

https://docs.laminas.dev/laminas-escaper/

BSD 3-Clause "New" or "Revised" License

191 stars 20 forks source link

Provide PHP 8.1 support #27

Closed weierophinney closed 3 years ago

weierophinney commented 3 years ago

This patch provides PHP 8.1 support, by making the following changes:

Adds 8.1 to the PHP constraint
Changes how we replace zend-escaper
- Renames "replace" to "conflict" in composer.json, and sets the constraint for "zend-escaper" to "*"
- Removes laminas-zendframework-bridge requirement
Adds Laminas CI configuration to ignore PHP platform req on PHP 8.1 builds

Fixes #13

weierophinney commented 3 years ago

This patch also resolves #13

weierophinney commented 3 years ago

Psalm seems to complain in this one:

Yep - it's a PHP 8-specific symfony/polyfill-mbstring file; it should be excluded from our scans (as it's in the vendor tree), but for some reason is still showing. I updated the baseline so that it gets ignored, and tests are now green.