Skip to content

Conversation

mvorisek
Copy link
Contributor

@mvorisek mvorisek commented Sep 27, 2024

needs #3488 to be merged first

Figuring out regexes that are not parsed by PHPStan grammar is hard. Let's help developers and they can help us.

With good reports PHPStan grammar can be improved quickly so all PCRE patterns are supported by PHPStan parser.

@Seldaek
Copy link
Contributor

Seldaek commented Sep 27, 2024

I'm scared to see what shows up, but that'd definitely be valuable.

@mvorisek mvorisek marked this pull request as ready for review September 27, 2024 11:54
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

@ondrejmirtes
Copy link
Member

Errors reported by PHPStan should be bugs in analysed code fixable by the developer.

Errors reported by PHPStan should not be PHPStan bugs and give more work to PHPStan users.

Feel free to use this rule in your own projects, and fix those bugs in PHPStan based on that.

@mvorisek mvorisek deleted the error_on_unparseable_const_regex branch November 29, 2024 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants