Skip to content
This repository has been archived by the owner on Apr 12, 2018. It is now read-only.

Config diff should be smarter to detect the tools #179

Open
stof opened this issue Apr 17, 2014 · 0 comments
Open

Config diff should be smarter to detect the tools #179

stof opened this issue Apr 17, 2014 · 0 comments

Comments

@stof
Copy link
Contributor

stof commented Apr 17, 2014

Currently, the diff is messy when a tool is disabled and another one is enabled because it does not consider a block as removed and the other block as added, but compares config of different tools together: https://scrutinizer-ci.com/g/Behat/Behat/inspections/1d22b986-1053-4a9b-9cd6-b24d8443d087/config/diff
This makes the diff totally useless (I'm quite sure the php_code_sniffer config is the same here).

and having all PHPCS rules rendered on a single line would make the diff useless as well when a rule is enabled, as it would mark the whole line as changed, forcing to diff by hand.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant