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
Make golden tests ignore new lines differences #4315
Comments
@kblok Interesting. We do run tests on Appveyour - and they run just fine. How can it be? |
It turns out that I have core.autocrlf in true. What seems to be the default in Windows. Long story short it will turn From now on it's up to you 😛
|
This would loosen the assertions, making it more "magical" - I don't like it
This sounds good! It'll also embrace a similar setup for all our contributors. Would you mind sending a PR? You might have a good idea on what to say in CONTRIBUTING.MD since you're going through this right now. |
Will do! |
Although we said on #4315 to add just a comment in the CONTRIBUTING.md file. I think this solution will be better for the repository's health. We could also apply this for `*.js` but it's up to you. This is what I needed to run after I added this file: ``` git rm --cached -r . git reset --hard ``` Fixes #4315
I have a few tests throwing errors because Chrome is returning
\r\n
instead of\n
on windows.How about changing
toBeGolden
to strip out new lines?The text was updated successfully, but these errors were encountered: