-
-
Notifications
You must be signed in to change notification settings - Fork 133
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
Cache exists error #41
Comments
does the error persist on a rerun? iirc there's an unavoidable race on initial cache population |
That could be the problem, then - it was the first run (and I dropped the token and it worked on the second run, so maybe it was just the rerun that fixed it). |
nice! yeah there isn't really anything we can do about that sadly |
https://pre-commit.ci will at least not have this problem |
Also got this error on a private repo but without additional features enabled. No error after job rerun. |
I love the new v2 action, but I got an odd error when converting a repo to use it:
The only think I've done differently (that I know of) is I enabled the token, just to see the guidelines in the pre-commit/action readme which say it only works on a private repository really means it only works on PRs from the same repository (which, while not as useful as working on PRs from forks, still would be useful in some cases).
https://github.com/scikit-hep/hepunits/pull/20/checks?check_run_id=950105310
The text was updated successfully, but these errors were encountered: