coding-standard icon indicating copy to clipboard operation
coding-standard copied to clipboard

Refine rules

Open blue32a opened this issue 3 years ago • 3 comments

Squiz.WhiteSpace.CastSpacing

This is included in PSR-12.

vendor/bin/phpcs --standard=PSR12 -e
Squiz (15 sniffs)
-----------------
  Squiz.Classes.ValidClassName
  Squiz.ControlStructures.ControlSignature
  Squiz.ControlStructures.ForEachLoopDeclaration
  Squiz.ControlStructures.ForLoopDeclaration
  Squiz.ControlStructures.LowercaseDeclaration
  Squiz.Functions.FunctionDeclaration
  Squiz.Functions.FunctionDeclarationArgumentSpacing
  Squiz.Functions.LowercaseFunctionKeywords
  Squiz.Functions.MultiLineFunctionDeclaration
  Squiz.Scope.MethodScope
  Squiz.WhiteSpace.CastSpacing
  Squiz.WhiteSpace.ControlStructureSpacing
  Squiz.WhiteSpace.ScopeClosingBrace
  Squiz.WhiteSpace.ScopeKeywordSpacing
  Squiz.WhiteSpace.SuperfluousWhitespace

PSR2.ControlStructures.ControlStructureSpacing

This is not included in PSR-12.

vendor/bin/phpcs --standard=PSR12 -e
PSR2 (9 sniffs)
---------------
  PSR2.Classes.ClassDeclaration
  PSR2.Classes.PropertyDeclaration
  PSR2.ControlStructures.ElseIfDeclaration
  PSR2.ControlStructures.SwitchDeclaration
  PSR2.Files.ClosingTag
  PSR2.Files.EndFileNewline
  PSR2.Methods.FunctionCallSignature
  PSR2.Methods.FunctionClosingBrace
  PSR2.Methods.MethodDeclaration

blue32a avatar Oct 03 '20 11:10 blue32a

This is not included in PSR-12.

Then how does it make PSR2.ControlStructures.ControlStructureSpacing.SpacingAfterOpenBrace unnecessary?

greg0ire avatar Apr 03 '21 10:04 greg0ire

Then how does it make PSR2.ControlStructures.ControlStructureSpacing.SpacingAfterOpenBrace unnecessary?

It is included in PSR2.ControlStructures.ControlStructureSpacing and is not installed. And it is checked by PSR12.ControlStructures.ControlStructureSpacing. Do you need it?

blue32a avatar Apr 10 '21 03:04 blue32a

@carusogabriel , you introduced that line, can you maybe chime in?

greg0ire avatar Apr 10 '21 07:04 greg0ire