-
Notifications
You must be signed in to change notification settings - Fork 55
pack command missing package-lock.json #50
Comments
I'm experiencing this issue as well. I tried setting the unchangeable flag on the package-lock.json file to see if that'd fix the issue, but it seems the package-lock.json is being renamed npm-shrinkwrap.json, which fails with the unchangeable flag.
|
Hello, This issue is du to prepack script (in A quick workaround is to restore ...
"prepack": "rm -rf lib && tsc && oclif-dev manifest && oclif-dev readme && npm shrinkwrap && git checkout -- package-lock.json",
... |
npm should use the shrinkwrap file in place of the lockfile though, no? |
Yes, npm use shrinkwrap for publishing module but the We need both files |
I am getting this error as well, on Windows. Seems we shouldn't be deleting |
I am getting this error as well. It is definitely |
🎉 This issue has been resolved in version 1.19.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Thanks for making this release. But it does not seem to fix the problem.
I even tried blowing away So now instead of Thanks! :-) |
Well, actually the error message says |
remove the bit in package.json that's deleting the shrinkwrap file and it should work after you run this is only required for existing CLIs |
The issue was me not understanding that |
Though dev-cli now will support either. |
@jdxcode do you have an answer to fix the above issue. $ oclif-dev -v $ git status $ oclif-dev pack:deb |
I've added |
Thank you @lordrip |
Environment
Linux Mint 18.3
Kernel 4.17.4
Node 8.11.3 (also tried 10.x)
NPM 5.10.0
oclif-dev 1.15.4
Problem
Whenever I execute
oclif-dev pack
(with npx or local) I getOf course I have a
package-lock.json
at the root of thecli
directory prior to executing the command, butpack
just deletes it and complains it's missing after.The text was updated successfully, but these errors were encountered: