Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

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

Successfully merging this pull request may close these issues.

3 participants