-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move back to npm #1402
Comments
As an example, in our "lodash": "4.17.5", This shows up in
In "lodash": {
"version": "4.17.5",
"resolved": "https://registry.npmjs.org/lodash/-/lodash-4.17.5.tgz",
"integrity": "sha512-svL3uiZf1RwhH+cWrfZn3A4+U58wbP0tGVTLQPbjplZxZ8ROD9VLuNgsRniTlLe7OlSqR79RUehXgpBW/s0IQw=="
}, What we want from the lock file is a way to go from our |
There is a definite performance loss with using npm versus yarn, but the tooling support is a bit easier under npm. |
@brettcannon are you planning on getting this done in this milestone. |
@DonJayamanne the plan is "when I find the time" 😉 |
While
yarn
is nice and fast, usingnpm
for itspackage-lock.json
will allow us to (hopefully) automate the generation of our third-party notices file more easily along with CELA/OSPO stuff behind the scenes.The key issue with
yarn.lock
is that it's a bespoke file format whilepackage-lock.json
is just JSON. One of those formats is a bit easier to read from Python. 😉 We will need to start usingnpm ci
where we useyarn install --freeze-lockfile
to make sure we are keeping the lock file updated.The text was updated successfully, but these errors were encountered: