-
Notifications
You must be signed in to change notification settings - Fork 30
Github Actions - Merge Queue #932
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
Conversation
I think we’ll need to tweak readthedocs as well? Did you take a look at what’s required there? |
@ngreenwald What did we need to tweak for readthedocs? It seems to be building properly. Is it the older package requirements that it uses? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it doesn't work with merge queues. Let's see how it goes.
@ngreenwald Tried to restart the CI workflow in the merge queue, however there was no response coming from the workflow request (probably b/c there's no space). After 12 min of no responses it kicked the PR off the merge queue. Alex's latest 2 run also did not start successfully. It's also odd that readthedocs doesn't build. |
I merged this in, so at least the GA part of the merge queue will be working. RTD doesn't know to look for a merge queue. There's a description in the link I posted in the issue about what to do for non-GA integration. Separately, we'll need to free up space on GA |
If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #931.
How did you implement your changes
Adds merge queue, removes python Dependabot.
Remaining issues
N/A.