-
Notifications
You must be signed in to change notification settings - Fork 735
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
Move from paths-filter #3808
Comments
dorny/paths-filter#108 Hmm I don't think it's paths-filter, I think we're hitting it somewhere else, and I think it's just the first thing to hit the API... |
Hello @edmundmiller, using dorny/paths-filter#133 I have been to alleviate the API rate limit issues we were hitting on https://github.com/vitessio/vitess. Here is the PR that uses dorny/paths-filter#133 in vitess: vitessio/vitess#10106 cc @mirpedrol |
@frouioui Hey thanks for reaching out! I think @mirpedrol solved it in #4202 |
FYI: the @frouioui fix has been merged in dorny/paths-filter#133 |
Have you checked the docs?
Description of the bug
It's served us well, but we're hitting API limits.
Maybe we're able to pass it an API key through a secret?
Example in #3807
Command used and terminal output
No response
Relevant files
No response
System information
No response
The text was updated successfully, but these errors were encountered: