-
Notifications
You must be signed in to change notification settings - Fork 125
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
Validate Auto-Version #389
Comments
This is deliberate, but unfortunate. We used to do this, but it results in a large number of requests and we would get rate limited. We could potentially add an |
Could the rate limit be worked around by instead creating PRs that get auto merged when the checks pass? |
The rate limiting was caused by the package downloads, I'm not sure how additional PRs would help? |
Apologies for the novice question, but I figured I'd just ask to be safe: if I play around with the workflows on my own fork to better understand the issue (replacing the github token with my own), that wouldn't affect this repository in any way would it? |
No, not at all. |
When you say rate limit, do you mean GitHub API rate limit, or is there some other limit?
I had assumed that the package sanity check included a |
Auto-Version does not validate that newly added versions can be installed and validated.
See: #388
It might be nice if the auto-version github workflow could catch errors like this, for example by attempting to install any auto-versioned packages.
The text was updated successfully, but these errors were encountered: