JakeQZ

Add a CI check for line length

1

The 120 character limit was enforced by PHP_CodeSniffer, but that was removed from the toolset in #1253 because it was not well-maintained and did not support PER-2. Perhaps an alternative...

... including custom properties like `--blink-182` Discovered in #1500, we have two separate regex matchers for these.

This was added in #1097, but is not yet employed. Also, it might be better namespaced in `Utilities` rather than `Caching`, as we are unlikely to have other caching-related classes.

Incorrect error when using `Closure::fromCallable` with `preg_replace_callback`

3

### Bug report https://phpstan.org/r/21523ebb-eb82-4d46-97fd-c1aa474a06f7 > Parameter #2 `$callback` of function `preg_replace_callback` expects `callable(array): string`, `Closure(array): string` given. [The documentation is not precise](https://www.php.net/manual/en/function.preg-replace-callback.php), but it can be determined from it that...

For example, `preg_split` should have its return type as `list` and not merely `list`. A more complex case is `preg_replace`, where the return type depends on the argument type. [PHPStan...

Parse selectors into component classes

2

See https://github.com/MyIntervals/PHP-CSS-Parser/pull/138#issuecomment-418193262 Adding this to the wishlist.

#389 and #1437 are closed because they are too global. Expressions as length values are not part of any standard, e.g. ```css .left { width: (10% + 0.5vw); } ```...

If it fails to parse selectors, it currently consumes and discards up to the next `}`. That should be the job of the caller, `CSSList`.

Minimum version of `composer-normalize` requires PHP 7.4

2

This is a PHIVE-installed pacakage. We could do one of two things: - Reduce the minimum version requirement in `phars.xml` to allow a version compatible with PHP 7.2; - Document...

Currently `parseListItem()` returns `null` to indicate that the end of the list has been reached, and `false` to indicate that an invalid item was found but parsing of the remaining...