doctrine / lexer

Base library for a lexer that can be used in Top-Down, Recursive Descent Parsers.
https://www.doctrine-project.org/projects/lexer.html
MIT License
11.05k stars 60 forks source link

Tell Psalm to use the lowest supported version #122

Closed greg0ire closed 4 months ago

greg0ire commented 4 months ago

Otherwise it is suggesting we do things that are not supported with PHP 8.1, like typing constants: https://php.watch/versions/8.3/typed-constants

greg0ire commented 4 months ago

Blocks https://github.com/doctrine/lexer/pull/121

derrabus commented 4 months ago

Let's ignore the MissingClassConstType error instead. Psalm errors are weirder if we test on a low language level with latest deps, so I can live with ignoring a few bogus error classes.

greg0ire commented 4 months ago

Done in https://github.com/doctrine/lexer/pull/123