Jaroslav Hanslík

icon indicating a website link https://twitter.com/kukulich

icon company Slevomat.cz icon location Neratovice, Czech Republic icon location Working in @slevomat by day, working on open source by night.

Results 108 comments of Jaroslav Hanslík

False negative SlevomatCodingStandard.ControlStructures.DisallowTrailingMultiLineTernaryOperator

It's intentional: https://github.com/slevomat/coding-standard/commit/f02675182fca6ba88e839a9433ebf4557e356015 The actual report matches the sniff name. Your version does make sense but it will need another sniff.

Better detection of unused variable

I expect I will remove this sniff in future major version. It is job more for PHPStan/Psalm.

Ignore unused variable with SplFileObject when setting it to NULL

Is not possible that `unset($sFo)` will work too?

How to pass empty array for property?

You can disable reporting specific error codes

Irresolvable conflict between DocCommentSpacing rules since 8.14.0

Cc @ondrejmirtes

`SlevomatCodingStandard.Commenting.DocCommentSpacing.IncorrectOrderOfAnnotationsGroup` Doesn't respect array shape comments

Not sure if it's possible to support this. It's not standard doccomment.

UnusedVariableSniff detects anonymous class property as unused variable

@mfb I'm not able to reproduce it. Can you please prepare failing test?

False positive SlevomatCodingStandard.Exceptions.RequireNonCapturingCatch.NonCapturingCatchRequired

Fixed in https://github.com/slevomat/coding-standard/commit/d82a6713c0063ba31e1d2bbc8a6df22798a2e370

`SlevomatCodingStandard.Namespaces.UnusedUses` cannot handle properties with (nullable) intersection types

I think PHPCS does not support https://wiki.php.net/rfc/dnf_types

Array declaration spacing sniff

I think it should be solved by standard sniffs because it's constructor call.