add phpcs to travis #936

Merged
merged 1 commit into from Nov 3, 2012

4 participants

@ceeram
CakePHP member

No description provided.

@lorenzo lorenzo merged commit c60237e into cakephp:master Nov 3, 2012
@shama

👏

@ceeram
CakePHP member

@shama are you sure about the thumbs up? Running phpcs with gitblame reports, blamed you for most cs errors ;)

@shama

@ceeram My phpcs doesn't say that. :)

PHP CODE SNIFFER GIT BLAME SUMMARY
--------------------------------------------------------------------------------
AUTHOR                                              (Author %) (Overall %) COUNT
--------------------------------------------------------------------------------
nate                                                    (0.08)     (32.26)    40
mark_story                                              (0.05)     (20.97)    26
Juan Basso                                              (0.22)     (11.29)    14
AD7six                                                  (0.07)      (8.87)    11
Jose Lorenzo Rodriguez                                  (0.07)      (5.65)     7
Mark Story                                              (0.05)      (4.03)     5
phpnut                                                  (2.96)      (4.03)     5
José Lorenzo Rodríguez                                 (0.06)      (3.23)     4
ADmad                                                    (0.5)      (2.42)     3
Kyle Robinson Young                                     (0.02)      (1.61)     2
Thomas Ploch                                            (0.15)      (1.61)     2
DarkAngelBGE                                            (2.13)      (0.81)     1
renan.saddam                                            (0.03)      (0.81)     1
TommyO                                                  (0.14)      (0.81)     1
Ceeram                                                  (0.02)      (0.81)     1
jperras                                                 (0.01)      (0.81)     1
--------------------------------------------------------------------------------
A TOTAL OF 124 SNIFF VIOLATION(S) WERE COMMITTED BY 16 AUTHOR(S)
--------------------------------------------------------------------------------
@ADmad
CakePHP member

Out of those 124 about 36 are "Code after RETURN statement cannot be executed" errors because of a sniff which incorrectly complains about return statements in switch / case blocks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment