chore: Add node/npm engines in package.json

[cicoub13]

https://docs.npmjs.com/cli/v6/configuring-npm/package-json

After discussion https://community.gladysassistant.com/t/fresh-development-installation/5785

[codecov]

Codecov Report

Merging #978 (125a389) into master (050c589) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #978   +/-   ##
=======================================
  Coverage   94.47%   94.47%           
=======================================
  Files         483      483           
  Lines        6405     6405           
=======================================
  Hits         6051     6051           
  Misses        354      354           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 050c589...125a389. Read the comment docs.

[VonOx]

Not sure if this help because no warning is produce and user can run anything ( Maybe I'm wrong ? )

[cicoub13]

Not sure if this help because no warning is produce and user can run anything ( Maybe I'm wrong ? )

You're right, no warning but developers can now easily which version of node/npm to use. If you think documentation (https://gladysassistant.com/fr/docs/api/setup-development-environnement) is enough, I'll close this PR.

There is one way to force node version, it's by adding a preinstall script in package.json that checks version. But I find it too restrictive.

[VonOx]

You're right, no warning but developers can now easily wich version of node/npm to use. If you think documentation (https://gladysassistant.com/fr/docs/api/setup-development-environnement) is enough, I'll close this PR.

No , I'm just asking ;)

I find it too restrictive.

I agree

[Pierre-Gilles]

Good idea!

[Pierre-Gilles]

@all-contributors please add @cicoub13 for code

[allcontributors]

@Pierre-Gilles

I've put up a pull request to add @cicoub13! 🎉