Support PHP 8.3 typed class constants #2396
Conversation
|
@mreiden The PR in the parser has landed in |
| @@ -14,6 14,7 @@ function parse(text, opts) { | |||
| const parser = new engine({ | |||
| parser: { | |||
| extractDoc: true, | |||
| version: opts.phpVersion, | |||
There was a problem hiding this comment.
This won't be needed once glayzzle/php-parser#1150 has landed.
There was a problem hiding this comment.
It should use the php version a user sets (https://github.com/prettier/plugin-php#configuration) instead of using the php-parser default though? If I remember correctly, the user setting has no effect on the parser without this line.
The php-parser default now being 8.3 makes it unnecessary for most people, but if someone set it to 8.0 it would use unsupported syntax. Then again, only 8.2 are currently supported by the php team, so maybe it's time to drop all the 5.x and 7.x versions and the default changed from "7.0" to "8.3"?
…d 7.x. This matches the default php version used in the php-parser project and removes options for very old eol versions. PHP 5.6 eol in January 2019 and 7.4 in November 2022.
|
@czosel this will need a new release of php-parser to include the fix to the original implementation. |
Depends on php-parser dependency support.
Kenneth-Sills/php-parser#1