fix: fix deprecated subdependencies found warn(#248)

[0x0bit]

Checklist

• [x] run npm run test and npm run benchmark
• [x] tests and/or benchmarks are included
• [x] documentation is changed or added
• [x] commit message and code follows the Developer's Certification of Origin and the Code of conduct

[mcollina]

Those are only devDependencies, they won't affect the actual warnings on install

[0x0bit]

Those are only devDependencies, they won't affect the actual warnings on install

Development dependencies are often installed and used during development, and by default, the npm install command installs all dependencies in the package.json file, including dependencies and devDependencies.

[mcollina]

@gurgunday ptal.

I think this should be rebased on the next branch.

[0x0bit]

What is the current status of this？

[gurgunday]

Taking a look

[Uzlopak]

Missing change in readme.md regarding code style button

[Uzlopak]

maybe take https://github.com/fastify/fastify/pull/5509 as reference

[climba03003]

by default, the npm install command installs all dependencies in the package.json file, including dependencies and devDependencies.

I don't think that is the truth as a dependencies in project.

mkdir repro
cd repro
npm install @fastify/helmet

You can see only dependencies is installed.

[gurgunday]

Interesting, normally it should also install devDeps

https://docs.npmjs.com/cli/v10/commands/npm-install#description

Maybe you have NODE_ENV set to production?

[climba03003]

Interesting, normally it should also install devDeps

npm install only install the devDependencies for the working project.

It never install devDependencies when you install package as a dependent.