Skip to content
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

fix infinite loop again #1364

Merged
merged 1 commit into from
Jul 3, 2023
Merged

fix infinite loop again #1364

merged 1 commit into from
Jul 3, 2023

Conversation

drammock
Copy link
Collaborator

closes #1359

needs upstream pradyunsg/sphinx-theme-builder#44 and will need a change to the commit hash in noxfile.py once that is merged.

@trallard
Copy link
Collaborator

Thank you for the time and effort put into fixing this!!!!
This is such an annoying bug 😤

@drammock drammock marked this pull request as ready for review July 3, 2023 16:09
@drammock
Copy link
Collaborator Author

drammock commented Jul 3, 2023

even though this doesn't fully fix the problem (needs pradyunsg/sphinx-theme-builder#44) I'm going to go ahead and merge it because at least then we can locally benefit from using the PR branch of the upstream project without having to manually tweak the noxfile every time. I'll get notified when pradyunsg/sphinx-theme-builder#44 is merged anyway so I won't easily forget to update the hash when the time comes.

@drammock drammock merged commit 815f6b7 into pydata:main Jul 3, 2023
@drammock drammock deleted the finite-loop branch July 3, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

infinite loop is back
2 participants