-
Notifications
You must be signed in to change notification settings - Fork 161
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
limit decorator compatible with asyncio #26
Comments
evemorgen
changed the title
limit decorator compatible with asyncio
Oct 29, 2018
limit
decorator compatible with asyncio
Any progress on this? |
Thanks @evemorgen - looks good. We've implemented this (for now) with https://github.com/hallazzang/asyncio-throttle and it works, but as the library isn't supported we'd rather move to a more mature project as https://github.com/tomasbasham/ratelimit. |
@tomasbasham would it be possible to merge this, now that python 2.7 is unsupported ? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I find this module very useful and promising - however more and more of my code is getting
asyncio
based. Wouldn't it be nice iflimit
decorator could beasyncio
compliant?It seems like
aio-limit
would be essentially the same aslimit
with couple ofasync
andawait
in it.I'm willing to make a PR for that. Would you guys consider merging such thing?
The text was updated successfully, but these errors were encountered: