nodejs-reference-architecture icon indicating copy to clipboard operation
nodejs-reference-architecture copied to clipboard

Periodic Review of Recommended Modules

Open lholmquist opened this issue 2 years ago • 4 comments

We want to start reviewing our current list:

  • still make sense ?
  • add any ?
  • remove any ?

https://github.com/nodeshift/nodejs-reference-architecture/blob/main/npcheck.json

lholmquist avatar Oct 24 '23 15:10 lholmquist

Should we add jose for jwt etc? node-fetch should be replaced by undici. Maybe axios too?

dbrugger avatar Oct 25 '23 06:10 dbrugger

Axios looks like is already there. We can add undici but I think just removing node-fetch should be enough, people can simply use the built-in fetch anyway.

As for jose, big +1 from me.

Since eslint is already there, I'd add prettier as well, considering these two go hand-in-hand as industry standards at this point.

FrivalszkyP avatar Jan 10 '24 16:01 FrivalszkyP

Since eslint is already there, I'd add prettier as well, considering these two go hand-in-hand as industry standards at this point.

Also ICYMI eslint has decided to deprecate formatting rules: https://eslint.org/blog/2023/10/deprecating-formatting-rules/

richardlau avatar Jan 10 '24 17:01 richardlau

Good call!

FrivalszkyP avatar Jan 10 '24 20:01 FrivalszkyP