289
78%
1.11: 82%

Ran 04 Jan 2015 04:08PM UTC

Jobs 5

Files 89

Run time 1min

Badge

Embed ▾

pending completion

Build # 289

Build Type

push

travis-ci

Committed by Dariusz Rumiński

Commit Message

bug #955 ParenthesisFixer - fix case with list call with trailing comma (keradus)

This PR was merged into the 1.4 branch.

Discussion
----------

ParenthesisFixer - fix case with list call with trailing comma

Fix #930
Clone #952

Consider following code:
`<?php list($path, $mode,) = foo();`

If we run MethodArgumentSpaceFixer on it, it will add a space after comma:
`<?php list($path, $mode, ) = foo();`

Then if we run ParenthesisFixer it will remove that space:
`<?php list($path, $mode,) = foo();`

This PR change behavior of ParenthesisFixer - stop removing space if there is trailing comma.
Also added a tests for it in both fixers.

Commits
-------

0db8e07 ParenthesisFixer - fix case with list call with trailing comma

Run Details

2658 of 3359 relevant lines covered (79.13%)

169.21 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	289.1	04 Jan 2015 04:07PM UTC	77.67	Travis Job 289.1
2	289.2	04 Jan 2015 04:07PM UTC	77.95	Travis Job 289.2
3	289.3	04 Jan 2015 04:07PM UTC	78.07	Travis Job 289.3
4	289.4	04 Jan 2015 04:07PM UTC	78.07	Travis Job 289.4
5	289.5	04 Jan 2015 04:07PM UTC	69.09	Travis Job 289.5

Source Files on build 289

Detailed source file information is not available for this build.

keradus / PHP-CS-Fixer / 289
78%
1.11: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 289

keradus / PHP-CS-Fixer / 289 78% 1.11: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 289

keradus / PHP-CS-Fixer / 289
78%
1.11: 82%

README BADGES
x