7
90%
1.11: 82%

Ran 06 Dec 2015 11:27PM UTC

Files 141

Run time 3s

Badge

Embed ▾

Committed 06 Dec 2015 10:50PM UTC coverage: 79.256% (+0.3%) from 78.966%

Job # 1100.7

Build Type

push

travis-ci

Committed by

keradus

Commit Message

feature #1493 Added MethodArgumentDefaultValueFixer (lmanzke)

This PR was submitted for the 1.11 branch but it was merged into the 1.12 branch instead (closes #1493).

Discussion
----------

Added MethodArgumentDefaultValueFixer

Hey guys, glad to be doing my first PR for you :).

My colleague @dereuromark opened an issue for a new fixer suggestion a while ago.
https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1462

Here is the PR implementing it. I took his solution and refactored it, please feel free to have a look.

I tried to make it as readable as possible.

Maybe you have some suggestion on how finding the next occurrence of a Variable OR certain token might be done more conveniently. I thought I could use

```
$tokens->getNextTokenOfKind(T_VARIABLE)
```

here, but it does not work that way ;).

Looking forward to your feedback!

Best regards,

Lucas

Commits
-------

a048e25 Added MethodArgumentDefaultValueFixer

Run Details

1555 of 1962 relevant lines covered (79.26%)

23.36 hits per line

keradus / PHP-CS-Fixer / 1100 / 7
90%
1.11: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1100.7

keradus / PHP-CS-Fixer / 1100 / 7 90% 1.11: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1100.7

keradus / PHP-CS-Fixer / 1100 / 7
90%
1.11: 82%

README BADGES
x