-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Conversation
hm.., it seems .php_cs config didn't effect the cs, still error : Running library/Zend/Db CS checks
Loaded config from ".php_cs"
..................F.............................................................................................................................................................................
Legend: ?-unknown, I-invalid file syntax, file ignored, .-no changes, F-fixed, E-error
1) Sql/Select.php (join_function) |
@Ocramius any idea ? |
@samsonasik run the same command with a |
it gets output with diff : Running library/Zend/Db CS checks
Loaded config from ".php_cs"
...........................................................................................................................................................................F....................
Legend: ?-unknown, I-invalid file syntax, file ignored, .-no changes, F-fixed, E-error
1) Sql/Select.php (join_function)
---------- begin diff ----------
--- Original
+++ New
@@ @@
*/
- public function join($name, $on, $columns = self::SQL_STAR, $type = self::JOIN_INNER)
+ public function implode($name, $on, $columns = self::SQL_STAR, $type = self::JOIN_INNER) as the exclude with ->notPath didn't work |
Meh, I'd rather remove that rule instead (buggy) |
the join -> implode issue itself patched in dev-master of php-cs-fixer PHP-CS-Fixer/PHP-CS-Fixer@830e64a . should just pointed to dev-master#commit instead ? |
@samsonasik I'd wait for a stable release instead |
(and revert that fixer rule for now) |
ok, rule reverted ;) |
@samsonasik weirdly, still failing |
Ah, sorry, you probably misunderstood what I was thinking of this rule |
maybe "join_function" rule? Warm regards, Abdul Malik Ikhsan Pada 30 Des 2014, pukul 01.58, Marco Pivetta notifications@github.com menulis:
|
Meh, was linking the wrong hash :-) https://github.com/zendframework/zf2/blob/ce693869bac0cef153227250d9a22a4d568ff0ff/.php_cs#L41 |
@Ocramius ok, done ;). "join_function" reverted. |
travis develop green now ;) |
Awesome, thanks! |
try fix travis cs develop again
…ev-cs-try-fix try fix travis cs develop again
…ev-cs-try-fix try fix travis cs develop again
latest travis develop built seems .php_cs config
->notPath('Zend/Db/Sql/Select.php')
didn't work, let's see now.