Skip to content

Commit

Permalink
Update NPM link to v10
Browse files Browse the repository at this point in the history
  • Loading branch information
chgeo authored Feb 5, 2024
1 parent c4a033c commit 83b861d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion node.js/best-practices.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ Therefore, the rules when publishing packages for reuse are:
* **Do** an *npm update* before publishing and test thoroughly.
(→ ideally automated in your CI/CD pipeline).
* **Do** the vulnerability checks for your software and all open-source software used by you **or by packages you used** (→ [Minimize Usage of Open Source Packages](#oss)).
* **Don't** do `npm shrinkwrap` → see also [npm's docs](https://docs.npmjs.com/cli/v7/configuring-npm/npm-shrinkwrap-json): *"It's discouraged for library authors to publish this file, ..."*
* **Don't** do `npm shrinkwrap` → see also [npm's docs](https://docs.npmjs.com/cli/v10/configuring-npm/npm-shrinkwrap-json): *"It's discouraged for library authors to publish this file, ..."*

::: tip
If both your package and a consuming package reuse the same CDS models, loading those models would fail because it's impossible to automatically merge the two versions, nor is it possible to load two independent versions. The reason for this is that it's reusing models that share the **same** single definitions.
Expand Down

0 comments on commit 83b861d

Please sign in to comment.