Skip to content

Remove line from "nikic/php-parser" error message#2845

Merged
ondrejmirtes merged 3 commits intophpstan:1.10.xfrom
mvorisek:dedup_line_nr_in_msg
Apr 22, 2024
Merged

Remove line from "nikic/php-parser" error message#2845
ondrejmirtes merged 3 commits intophpstan:1.10.xfrom
mvorisek:dedup_line_nr_in_msg

Conversation

@mvorisek
Copy link
Contributor

@mvorisek mvorisek commented Jan 1, 2024

related discussion/issue: phpstan/phpstan#10358 (comment)

@mvorisek
Copy link
Contributor Author

mvorisek commented Jan 1, 2024

Please advise where can I put https://phpstan.org/r/11e0c7af-bd52-4add-8de0-98397c5d492e test and on what test can I base it - I need to assert the phpstan error message.

@ondrejmirtes
Copy link
Member

AnalyserIntegrationTest

@mvorisek mvorisek marked this pull request as ready for review January 1, 2024 16:20
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

@mvorisek mvorisek marked this pull request as draft April 22, 2024 08:42
@mvorisek mvorisek force-pushed the dedup_line_nr_in_msg branch from 4468bec to fa7c4c5 Compare April 22, 2024 08:46
@mvorisek mvorisek force-pushed the dedup_line_nr_in_msg branch from 73c1127 to 084ca63 Compare April 22, 2024 09:14
@mvorisek mvorisek marked this pull request as ready for review April 22, 2024 09:16
@mvorisek mvorisek requested a review from ondrejmirtes April 22, 2024 09:16
@phpstan-bot
Copy link
Collaborator

This pull request has been marked as ready for review.

@ondrejmirtes ondrejmirtes merged commit 83ccb7f into phpstan:1.10.x Apr 22, 2024
@ondrejmirtes
Copy link
Member

Thank you!

@mvorisek mvorisek deleted the dedup_line_nr_in_msg branch April 22, 2024 11:02
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.

3 participants