-
Notifications
You must be signed in to change notification settings - Fork 114
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
Improve Travis configuration and add PHP-CS-Fixer #160
Conversation
8d7e1fc
to
96ec9d9
Compare
9e92b70
to
007f9a3
Compare
*/ | ||
public static function lookupDecimal($int) | ||
{ | ||
$entity = '&#' . $int . ';'; | ||
$entity = '&#'.$int.';'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arent spaces enforced when concatenating variables?
src/HTML5/Parser/Scanner.php
Outdated
@@ -284,12 +285,12 @@ public function charsWhile($mask) | |||
* | |||
* Newlines are column 0. The first char after a newline is column 1. | |||
* | |||
* @return int The column number. | |||
* @return int the column number |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there some rule that enforces the first letter lowercase?
What about adding a <?php
$finder = PhpCsFixer\Finder::create()
->in(__DIR__)
;
return PhpCsFixer\Config::create()
->setRules([
'@Symfony' => true,
'concat_space' => ['spacing' => 'one'],
])
->setFinder($finder)
; As a personal preference (and less changes to the current codebase), I suggest |
007f9a3
to
d829a30
Compare
6d7839a
to
91e7977
Compare
Updated :) . |
Thanks! |
No description provided.