[Feature] Diff output configurable #482
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
👋
This PR to improve display for diff.
By default, I add 1 line of context around the diff. I also improve the default display by removing some empty lines.
Before:

After:

But, now, we can configure the diff context around.
In
phpinsights.php
, add the following line:Output:
While testing, I noticed that when configuration file has some invalid options, the output was really ugly:
I rework a little this part to improve output when there is an invalid configuration: