bolt-python icon indicating copy to clipboard operation
bolt-python copied to clipboard

Add Quart framework adapter

Open sonnens opened this issue 2 years ago • 4 comments

want to support the asyncio loop in quart apps

sonnens avatar Mar 20 '22 05:03 sonnens

@sonnens Thanks for submitting this suggestion!

For others, more information about Quart framework:

  • https://pgjones.gitlab.io/quart/
  • https://gitlab.com/pgjones/quart
  • https://pypi.org/project/quart/

Before discussing the details of the implementation of the PR, let me share our policies on the built-in adapters.

  • We need unit tests and simple examples for long-term maintenance (I will let you know the details in the PR comment)
  • In general, we don't prioritize less popular frameworks (meaning less than 10K downloads per day) for built-in ones

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.

seratch avatar Mar 20 '22 06:03 seratch

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 avatar Oct 28 '22 04:10 seratch

@seratch any update here??

muhammad-ahsan099 avatar Jan 11 '24 11:01 muhammad-ahsan099

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.

seratch avatar Jan 11 '24 11:01 seratch