-
Notifications
You must be signed in to change notification settings - Fork 59
fix: line endings are now respected on windows #260
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
Conversation
🦋 Changeset detectedLatest commit: 0d39f6d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
a953456
to
3b5e991
Compare
Co-authored-by: Mateusz Burzyński <[email protected]>
05fa860
to
18f2fee
Compare
@Andarist Does everything look good now that I've added tests? |
@Andarist just a reminder - I made all the requested changes 😄 |
I'm sorry, I missed the previous notification ;p I was not holding this PR a hostage, I swear! I appreciate you paying the ransom though /j 😉 |
😆 I saw the projects you work on felt inclined to give something. I miss 99% of my GitHub notifications so I understand! |
On windows all the package.json files would get their line endings changed to lf. This would cause all the package.json files to be marked as modified. This pr fixes the issue by detecting the line endings of the package.json file and normalizes them back what they originally were
Closes #257