-
Notifications
You must be signed in to change notification settings - Fork 245
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
Add Quart framework adapter #619
Comments
@sonnens Thanks for submitting this suggestion! For others, more information about Quart framework: Before discussing the details of the implementation of the PR, let me share our policies on the built-in adapters.
Our rough standard to decide what framework to support is how widely a framework is accepted in the communities. We know that the number of PyPI package downloads is not the only metric representing popularity but we use the data as a criteria for our internal decision. The lowest level of the number for assessing the popularity is 10K per day. All the frameworks currently supported are above 10K (at least, they were above 10K when we decided to support). According to https://pypistats.org/packages/quart, Quart is downloaded 3.5K times per day as of this month. We have to say that this is not our high priority. If you need this adapter addition as early as possible, we as the maintainers of bolt-python and other Slack SDKs need your or community members' help. For the above reasons, I will ask you to put more efforts on your pull request. It'd be greatly appreciated if you could understand this. |
Hi @sonnens, I've revisited this and found that Quart is now more popular and its download per day is above 13K. If you are still interested in continuing with your PR, we are happy to review it. As I mentioned in the PR, we need unit tests and CI builds for all built-in adapters. Once these are added, your PR will be merged. Let us know what you think 👋 |
@seratch any update here?? |
We don’t have immediate plans to add the adapter. If you are in a hurry, please reuse the code in the PR. I haven’t verified but perhaps it works for you. There is no need to wait for a built-in support. |
want to support the asyncio loop in quart apps
The text was updated successfully, but these errors were encountered: