Skip to content
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

Add throttling to tarball requests #1

Closed
sebmck opened this issue Jan 19, 2016 · 0 comments
Closed

Add throttling to tarball requests #1

sebmck opened this issue Jan 19, 2016 · 0 comments

Comments

@sebmck
Copy link
Contributor

sebmck commented Jan 19, 2016

These don't go through the request manager (but probably should).

@sebmck sebmck closed this as completed Jul 13, 2016
pfremm-NM pushed a commit to northwesternmutual/yarn that referenced this issue Mar 13, 2017
mathagician pushed a commit to mathagician/yarn that referenced this issue Oct 21, 2017
rally25rs pushed a commit to rally25rs/yarn that referenced this issue Dec 10, 2017
feat(resolution) When using a local git repo, use `git show-ref` instead of `ls-remote`(yarnpkg#5017)
rally25rs added a commit to rally25rs/yarn that referenced this issue Apr 15, 2018
Previously yarn would allow a dependency of a github pullrequest with "yarnpkg#1/head" where 1 is the PR
number. npm allows the format "#pull/1/head" as well. For compatability, allowing yarn to fall back
to this format.
BYK pushed a commit that referenced this issue Apr 15, 2018
…5673)

**Summary**

Fixes #1610. Previously yarn would allow a dependency on a GitHub pull request with "#1/head" where 1 is the PR number. npm allows the format "#pull/1/head" as well. For compatibility, allowing Yarn to fall back to this format.

**Test plan**

Added a test to `git-ref-resolver` to test the case `"pull/10/head"`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant