Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use error identifiers from baseline (1.11) #148

Open
ruudk opened this issue Oct 31, 2023 · 3 comments
Open

Use error identifiers from baseline (1.11) #148

ruudk opened this issue Oct 31, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@ruudk
Copy link

ruudk commented Oct 31, 2023

In the upcoming PHPStan 1.11 we'll see // identifier: comments. Could this be useful to further analyze the baseline?

phpstan/phpstan-src@4b32cac

@staabm
Copy link
Owner

staabm commented Oct 31, 2023

I thought about that. It will definitely have an impact on this lib and its feature set.

Some things which are currently build here will be possible natively in a similar way.

I am open for discussion

@ruudk
Copy link
Author

ruudk commented Oct 31, 2023

The reason why I brought it up was that I just tried this package and got this result:

Analyzing phpstan-baseline.php
  Date: Tue, 31 Oct 2023 08:45:30 +0000
  Overall-Errors: 11957
  Classes-Cognitive-Complexity: 0
  Deprecations: 299
  Invalid-Phpdocs: 66
  Unknown-Types: 39
  Anonymous-Variables: 0
  Native-Property-Type-Coverage: 0
  Native-Param-Type-Coverage: 0
  Native-Return-Type-Coverage: 0
  Unused-Symbols: 0

Not very helpful as everything is marked as Overall-Errors. 😅 I hope that error identifiers will improve this list.

@staabm
Copy link
Owner

staabm commented Oct 31, 2023

it depends on the rules you use. atm we count only those errors emitted by the rules mentioned in the readme.

but I agree, we could aggregate errors by native identifiers

@staabm staabm added the enhancement New feature or request label Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants