Skip to content

Respect GitHub ratelimit to support more than 8 stacked PRs#289

Closed
bdaz wants to merge 2 commits intoezyang:masterfrom
thoughtful-inc:master
Closed

Respect GitHub ratelimit to support more than 8 stacked PRs#289
bdaz wants to merge 2 commits intoezyang:masterfrom
thoughtful-inc:master

Conversation

@bdaz
Copy link
Copy Markdown
Contributor

@bdaz bdaz commented Apr 24, 2025

Got tired of using the macOS Network Link Conditioner and --force to workaround this :)

Fixes: #267

I don't write a lot of Python, so let me know if anything seems off or weird here. But it seems to work for me.

@ezyang
Copy link
Copy Markdown
Owner

ezyang commented Apr 26, 2025

I probably can just fix the tests myself

@ezyang
Copy link
Copy Markdown
Owner

ezyang commented Apr 27, 2025

bah I don't have write permissions to the branch

@ezyang
Copy link
Copy Markdown
Owner

ezyang commented Apr 27, 2025

Fixed at #290

@ezyang
Copy link
Copy Markdown
Owner

ezyang commented Apr 27, 2025

merged as #290

@ezyang ezyang closed this Apr 27, 2025
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.

Exponential Backoff instead of "cowardly limit" of 8 stacked PRs?

2 participants