-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add a trailing newline to package.json when appropriate #17
Conversation
Co-authored-by: Angus Hollands <goosey15@gmail.com>
Hmm, not sure why the actions aren't running, there is no current GitHub incident, supposedly. |
Ah, they're set to only run on push. |
Mind if I tweak it to run on PRs to master as well? |
Haha, we're making the same change! You're welcome to do that, I'll hold off. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the tests pass, this gets my approval! :)
I copied what we have in |
Do you want to make a new release? I'm happy to, but you're welcome to ;) |
I'm happy to release, if you add |
Oh! I had thought that I'd already done this for you an @fcollonval. Done. |
Released! |
Thanks @agoose77 |
This was causing failures in maintainer-tools, which uses the
end-of-file-fixer
pre-commit hook.Npm and Yarn also typically add a newline character to the file when interacting with it.