Skip to content

Commit

Permalink
[PHPStan] Clean up PHPStan ignore errors config (#805)
Browse files Browse the repository at this point in the history
  • Loading branch information
samsonasik committed Aug 31, 2021
1 parent 93f0232 commit f74b7f6
Showing 1 changed file with 0 additions and 32 deletions.
32 changes: 0 additions & 32 deletions phpstan.neon
Original file line number Diff line number Diff line change
Expand Up @@ -309,8 +309,6 @@ parameters:
# PHP 7_4 literal syntax
- '#Property PhpParser\\Node\\Scalar\\DNumber\:\:\$value \(float\|int\) does not accept string#'

- '#Unable to resolve the template type T in call to method Rector\\NodeNestingScope\\ParentFinder\:\:findByTypes\(\)#'

# mimics original doctrine/annotations parser, improve later when finished
-
message: '#Array destruct is not allowed\. Use value object to pass data instead#'
Expand Down Expand Up @@ -408,16 +406,9 @@ parameters:
message: '#Function "dump_node\(\)" cannot be used/left in the code#'
path: src/functions/node_helper.php

-
message: '#Use separate function calls with readable variable names#'
path: 'src/FileSystem/PhpFilesFinder.php'

# union false positive
- '#Method Rector\\Comments\\CommentRemover\:\:removeFromNode\(\) has parameter \$node with no value type specified in iterable type array#'

# should be refactored to collector
- '#Cognitive complexity for "Rector\\CodingStyle\\Naming\\NameRenamer\:\:renameNameNode\(\)" is 10, keep it under 9#'

-
message: '#Class with base "StaticTypeMapper" name is already used in "Rector\\PHPStanStaticTypeMapper\\TypeMapper\\StaticTypeMapper", "Rector\\StaticTypeMapper\\StaticTypeMapper"\. Use unique name to make classes easy to recognize#'
path: packages/StaticTypeMapper/StaticTypeMapper.php #31
Expand All @@ -436,17 +427,6 @@ parameters:
message: '#Cannot return include_once/require_once#'
path: rules/Renaming/Rector/Name/RenameClassRector.php

# required assigns - parent looping
-
message: '#foreach\(\.\.\.\), while\(\), for\(\) or if\(\.\.\.\) cannot contains a complex expression\. Extract it to a new variable assign on line before#'
paths:
- packages/NodeNestingScope/FlowOfControlLocator.php
- src/PhpParser/Node/BetterNodeFinder.php
- rules/Php70/Rector/FuncCall/MultiDirnameRector.php
- src/Application/FileProcessor.php
- rules/CodingStyle/Rector/Assign/ManualJsonStringToJsonEncodeArrayRector.php
- rules/CodeQuality/Rector/Return_/SimplifyUselessVariableRector.php

# class-string miss match
- '#Parameter \#1 \$classes of method Rector\\BetterPhpDocParser\\PhpDocInfo\\PhpDocInfo<PHPStan\\PhpDocParser\\Ast\\Node\>\:\:getByAnnotationClasses\(\) expects array<class\-string\>, array<int, string\> given#'
- '#Parameter \#1 \$classes of method Rector\\BetterPhpDocParser\\PhpDocInfo\\PhpDocInfo<TNode of PHPStan\\PhpDocParser\\Ast\\Node\>\:\:getByAnnotationClasses\(\) expects array<class\-string\>, array<string\> given#'
Expand All @@ -455,12 +435,6 @@ parameters:
# weird generics
- '#Method Rector\\BetterPhpDocParser\\PhpDocInfo\\PhpDocInfo\:\:getByType\(\) should return array<TNode of PHPStan\\PhpDocParser\\Ast\\Node\> but returns array<PHPStan\\PhpDocParser\\Ast\\Node\>#'

# should be allowed, as continue check - fix in symplify
-
message: '#foreach\(\.\.\.\), while\(\), for\(\) or if\(\.\.\.\) cannot contains a complex expression\. Extract it to a new variable assign on line before#'
paths:
- packages/NodeNestingScope/ParentFinder.php

- '#Callable callable\(PHPStan\\Type\\Type\)\: PHPStan\\Type\\Type invoked with 2 parameters, 1 required#'
- '#Method Rector\\NodeNestingScope\\ParentFinder\:\:findByTypes\(\) should return T of PhpParser\\Node\|null but returns class\-string<T of PhpParser\\Node\>\|T of PhpParser\\Node#'

Expand Down Expand Up @@ -544,16 +518,10 @@ parameters:
paths:
- rules/PSR4/FileRelocationResolver.php
- rules/Privatization/Rector/Class_/RepeatedLiteralToClassConstantRector.php
- rules/Php74/Rector/LNumber/AddLiteralSeparatorToNumberRector.php

-
message: '#Parameter \#2 \$length of function str_split expects int<1, max\>, int given#'
paths:
- rules/Php74/Rector/LNumber/AddLiteralSeparatorToNumberRector.php

- '#^Cognitive complexity for "Rector\\CodingStyle\\Naming\\NameRenamer\:\:renameNameNode\(\)" is 13, keep it under 9$#'

-
message: '#The string value "" is repeated 6 times\. Refactor to enum to avoid typos and make clear allowed value#'
paths:
- packages/BetterPhpDocParser/PhpDocManipulator/PhpDocTagRemover.php #45

0 comments on commit f74b7f6

Please sign in to comment.