Skip to content

Commit

Permalink
Fix test-comment, boyscout priority-comments
Browse files Browse the repository at this point in the history
  • Loading branch information
dmvdbrugge committed Nov 22, 2018
1 parent 69003f1 commit 5655749
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/Fixer/Basic/BracesFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ public function bar($baz)
*/
public function getPriority()
{
// should be run after the ElseIfFixer, NoEmptyStatementFixer and NoUselessElseFixer
// should be run after the ElseIfFixer, LineEndingFixer, NoEmptyStatementFixer and NoUselessElseFixer
return 33;
}

Expand Down
2 changes: 2 additions & 0 deletions src/Fixer/ControlStructure/ElseifFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,8 @@ public function getDefinition()
*/
public function getPriority()
{
// Should run after NoAlternativeSyntaxFixer
// Should run before BracesFixer
return 34;
}

Expand Down
1 change: 1 addition & 0 deletions src/Fixer/ControlStructure/NoAlternativeSyntaxFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ public function isCandidate(Tokens $tokens)
*/
public function getPriority()
{
// Should run before BracesFixer and ElseifFixer
return 35;
}

Expand Down
1 change: 1 addition & 0 deletions src/Fixer/Whitespace/ArrayIndentationFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ public function isCandidate(Tokens $tokens)
public function getPriority()
{
// should run after BracesFixer
// should run before AlignMultilineCommentFixer and BinaryOperatorSpacesFixer
return 32;
}

Expand Down
2 changes: 1 addition & 1 deletion src/Fixer/Whitespace/LineEndingFixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ public function getDefinition()

public function getPriority()
{
// Should run before braces
// Should run before BracesFixer
return 34;
}

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
--TEST--
Integration of @Symfony [whitespace version].
Integration of fixers: line_ending,braces.
--RULESET--
{"line_ending": true, "braces": true}
--CONFIG--
Expand Down

0 comments on commit 5655749

Please sign in to comment.