-
Notifications
You must be signed in to change notification settings - Fork 191
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
Fix git checkout fallback, make ls-remote filtering stricter #364
base: main
Are you sure you want to change the base?
Conversation
@tschaub, could you take a look? |
@tschaub |
@antongolub - I'll admit it isn't clear to me what situation fails before this change and succeeds after. Having a test that demonstrates what this fixes would be ideal. I understand that might be hard to put together though. Could you try to put together a test? Or if not, could you provide steps for someone else to reproduce the problem that this fixes? |
The current check makes no diff between tag and branch. if we just add Example repo: https://github.com/antongolub/yarn-audit-fix
git ls-remote . origin/gh-pages
|
I was looking for something more like the steps required to reproduce the problem you are trying to solve. |
Scenario
|
PR is an attempt to fix #349