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

Add a trailing newline to package.json when appropriate #17

Merged
merged 3 commits into from
Nov 14, 2022

Conversation

blink1073
Copy link
Collaborator

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.

@blink1073 blink1073 added the bug Something isn't working label Nov 14, 2022
Co-authored-by: Angus Hollands <goosey15@gmail.com>
@blink1073
Copy link
Collaborator Author

Hmm, not sure why the actions aren't running, there is no current GitHub incident, supposedly.

@blink1073
Copy link
Collaborator Author

Ah, they're set to only run on push.

@blink1073
Copy link
Collaborator Author

Mind if I tweak it to run on PRs to master as well?

@agoose77
Copy link
Owner

agoose77 commented Nov 14, 2022

Haha, we're making the same change! You're welcome to do that, I'll hold off.

Copy link
Owner

@agoose77 agoose77 left a 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! :)

@blink1073
Copy link
Collaborator Author

I copied what we have in jupyter_server.

@blink1073 blink1073 merged commit 246b692 into agoose77:master Nov 14, 2022
@blink1073 blink1073 deleted the ensure-newline branch November 14, 2022 12:51
@agoose77
Copy link
Owner

Do you want to make a new release? I'm happy to, but you're welcome to ;)

@blink1073
Copy link
Collaborator Author

I'm happy to release, if you add blink1073 to the list of maintainers for https://pypi.org/project/hatch-nodejs-version/

@agoose77
Copy link
Owner

Oh! I had thought that I'd already done this for you an @fcollonval. Done.

@blink1073
Copy link
Collaborator Author

Released!

@fcollonval
Copy link
Collaborator

Oh! I had thought that I'd already done this for you an @fcollonval. Done.

Thanks @agoose77

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants