Skip to content
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

[test] Fix failing tests on Windows #22369

Merged
merged 3 commits into from Aug 26, 2020
Merged

[test] Fix failing tests on Windows #22369

merged 3 commits into from Aug 26, 2020

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Aug 26, 2020

Not sure if this is already fixed upstream but in case it's not @merceyz

Tests failed for me as well with core.autocrlf=false on win10 and passed after this fix.

@mnajdova Could you check this out and see if it works? If not please include the output of git config --get core.autocrlf

@eps1lon eps1lon added the test label Aug 26, 2020
@mui-pr-bot
Copy link

mui-pr-bot commented Aug 26, 2020

No bundle size changes comparing 3da0347...d7b7eee

Generated by 🚫 dangerJS against d7b7eee

@eps1lon eps1lon changed the title [test] Fix ttp tests failing on windows [test] Fix failing tests on Windows Aug 26, 2020
@eps1lon eps1lon requested a review from mnajdova August 26, 2020 13:55
@merceyz
Copy link
Member

merceyz commented Aug 26, 2020

Not sure if this is already fixed upstream but in case it's not @merceyz

It isn't but that repo is set up to always use lf line endings (https://github.com/merceyz/typescript-to-proptypes/blob/b713f56f5600a3cbcf55cb7c9384780fe09bbd54/.gitattributes#L1) so it shouldn't be a problem, i'll fix it anyways though. Cheers

@eps1lon
Copy link
Member Author

eps1lon commented Aug 26, 2020

It isn't but that repo is set up to always use lf line endings (merceyz/typescript-to-proptypes@b713f56/.gitattributes#L1) so it shouldn't be a problem, i'll fix it anyways though. Cheers

Then you could just remove that normalization. I don't think the intention was to only normalize the first linefeed.

@eps1lon eps1lon merged commit bf930d0 into mui:next Aug 26, 2020
@eps1lon eps1lon deleted the test/ttp-win branch August 26, 2020 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants