-
Notifications
You must be signed in to change notification settings - Fork 16
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 missing ITs for JSON Cadence parser #35
Conversation
The tests ran locally but seems to fail in the CI here. I approved in the meanwhile. |
@franklywatson yes, I noticed that the tests running against the network tend to throw "max rate exceeded" after running through the CI pipeline 2-3 times. I noticed the rate limit usually resets after 12-24 hours and then they are able to run again normally (just retriggered on this branch and they ran successfully)... not sure if we could do anything on our side to remove the request limit (could it be tied to the IP address of the Github VM used to run the tests?) |
Description
For contributor use:
master
branchFiles changed
in the Github PR explorer