-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TODO: Ability to ignore blank line changes #82
Comments
That is a good idea. I did spent any time on that yet. |
I guess it should no be too hard to implement in function linesAreDifferent. White space ignore is already there. <?php
preg_replace('/(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+/', '$1', $input); or better use
|
I don't know... I actually wanted you to clarify what the todo item means. 😅 |
it means, that in a comparison blank lines should be ignored (as an option). It is in the readme file before I forked the repo. |
I'm sorry, but I still don't follow 😅... Please provide an example so I can clearly understand what it means. |
Don't compare blank lines at all and or example file a) blank lines 10 - 12. file b) blank lines 10 - 27. With the option set there is no difference. For example when I compare CSV files I don't care about some blank lines. The content it what counts. Blank lines are ignored at import into any system. |
Nope. Sorry, I just don't get it. |
I created a gist. https://gist.github.com/JBlond/1632bc8b134a3dbb9545bbbdbcfe7021 |
Do you prefer to introduce a new tag, e.g. |
Choose as you like. Add a new tag is fine. It might be more clear. |
I think I'm getting somewhere. |
Please test PR #86 |
I'll get to that after Christmas. |
Can you elaborate on the ability to ignore blank line changes?
The text was updated successfully, but these errors were encountered: