Skip to content

Conversation

@techvlad
Copy link

@techvlad techvlad commented Jul 15, 2024

Enables rule and enforces style for future contributions https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/prefer-node-protocol.md

Initial work was done here

Example how it works in VSCode:
image

Possible improvements:

Comment on lines +61 to +63
"engines": {
"node": "^18 || >=20"
},
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my understanding pino supports only next node versions: ^18 || >=20
Required by eslint-plugin-n package

@techvlad techvlad marked this pull request as ready for review July 15, 2024 14:38
@jsumners
Copy link
Member

I expect that this project will switch to using https://www.npmjs.com/package/neostandard

@mcollina
Copy link
Member

Yes, exactly

@techvlad
Copy link
Author

ok, got it, thank you!

@techvlad techvlad closed this Jul 16, 2024
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants