• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

keradus / PHP-CS-Fixer / 2675
82%
1.11: 82%

Build:
Build:
LAST BUILD BRANCH: master
DEFAULT BRANCH: 1.11
Ran 21 Apr 2017 06:47AM UTC
Jobs 1
Files 269
Run time 14s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
2675

push

travis-ci

keradus
bug #2711 FixCommand - fix diff optional value handling (keradus)

This PR was merged into the 2.2 branch.

Discussion
----------

FixCommand - fix diff optional value handling

regression fix for #2554:
```php
new InputOption('diff', '', InputOption::VALUE_OPTIONAL, 'Also produce diff for each file.', 'sbd'),
```
that means:
- 'diff' option may or may not be passed
- when it's passed, value for it is optional
- 'sbd' is default value of `diff` option regardless it is passed or not (!!!)

I guess assumption during creating 2254 was that `sbd` would be only a value for `diff` if one would pass `--diff` (without value). Apparently, it is a value even if one would not pass that option at all, leading to generate diff even without using `--diff` option.

ref https://github.com/symfony/symfony/issues/11572

Commits
-------

5546893d FixCommand - fix diff optional value handling

7651 of 8827 relevant lines covered (86.68%)

22.94 hits per line

Jobs
ID Job ID Ran Files Coverage
1 2675.1 (DEPLOY=yes TASK_TESTS_COVERAGE=1) 21 Apr 2017 06:39AM UTC 0
86.68
Travis Job 2675.1
Source Files on build 2675
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #2675
  • 6820d96f on github
  • Prev Build on 2.2 (#2667)
  • Next Build on 2.2 (#2676)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc