-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Specifying hash in the package json is not working #3109
Comments
Also, if I delete my yarn.lock file and try |
If I update my package.json to the following, it works locally, but fails in the CI with a log "Couldn't find the binary git". This seems related to #2083.
|
Seems to work as expected if we use Does that help? |
It should work the same regardless of the protocol: git, git+https or https. We observe the same issue in our repository and we cannot use git protocol. |
One reason you might prefer to use https:// rather than git:// urls in this situation is that git:// URLs only work with github if you are on a machine with valid github SSH keys. Github allows anonymous access but only through https:// . Please correct me if I am wrong. |
Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
I have a stable branch in my project that gets updated every time the CI build passes.
Eventually we want to be able to use that branch in our package.json, but I'm facing yarn issues with bad hash. See this other related issue: #1916.
So to try and fix the hash issue we decided to specify the commit hash in the package.json itself, but yarn seems to be ignoring that entry.
If the current behavior is a bug, please provide the steps to reproduce.
Here is our
package.json
entry:Here is the
yarn.lock
section:What is the expected behavior?
You can see that the yarn.lock resolved to latest hash in my stable branch, where I was expecting it to resolve to the hash that I've specified.
Please mention your node.js, yarn and operating system version.
Node: v7.3.0
NPM: 3.10.10
Yarn: 0.21.3
The text was updated successfully, but these errors were encountered: