-
-
Notifications
You must be signed in to change notification settings - Fork 81
feat: add .nvmrc and engines
key to package.json
#66
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
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.
This looks great in general (thanks for sending it! ✨), but it's failing a lint task:
$ npx eslint . --max-warnings 0 --report-unused-disable-directives
template-typescript-node-package/package.json
Error: 51:2 error Expected object keys to be in ascending order. 'engines' should be before 'version' jsonc/sort-keys
This is one of the lint rules I've been meaning to bring into my other repos 😄
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.
💯 thanks!
Thanks @JoshuaKGoldberg! Would it be worth adding to a precommit hook locally? I personally have a tendency to "commit and forget." |
# [1.2.0](v1.1.2...v1.2.0) (2022-12-02) ### Features * add .nvmrc and `engines` key to package.json ([#66](#66)) ([db9a536](db9a536))
🎉 This PR is included in version 1.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
What would the precommit hook do? I'm wary of running ESLint in a hook because with type information it can be a little slow. But if there's a good way to only run untyped + fixable rules, that'd be nice... |
I was just thinking it could lint. 🤷🏻 I've used |
If a benchmark shows it's reasonably quick, I'm down! We'll want to get a meaty project in. |
@allcontributors add @Pinjasaur for tooling (whoops, sorry, I forgot I'd set this up!) |
I've put up a pull request to add @Pinjasaur! 🎉 |
PR Checklist
Overview
It's like JoshuaKGoldberg/refined-saved-replies#69 but for this repo. 🙃