Skip to content

Add Docker samples#52

Merged
seratch merged 6 commits into
slackapi:mainfrom
seratch:docker-samples
Sep 1, 2020
Merged

Add Docker samples#52
seratch merged 6 commits into
slackapi:mainfrom
seratch:docker-samples

Conversation

@seratch

@seratch seratch commented Aug 30, 2020

Copy link
Copy Markdown
Contributor

This pull request adds a few Docker samples.

  • google_cloud_run (Flask/Sanic/AIOHTTP app running on Google Cloud Run)
  • docker/aiohttp (built-in AIOHTTP based async/await app)
  • docker/fastapi-gunicorn (FastAPI/Starlette app running on Gunicorn/Uvicorn server)
  • docker/flask-gunicorn (Flask app running on Gunicorn server)
  • docker/flask-uwsgi (Flask app running on uWSGI server)
  • docker/sanic (Sanic app running its own server)

Category (place an x in each of the [ ])

  • slack_bolt.App and/or its core components
  • slack_bolt.async_app.AsyncApp and/or its core components
  • Adapters in slack_bolt.adapter

Requirements (place an x in each [ ])

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/run_tests.sh after making the changes.

@seratch seratch added the docs Improvements or additions to documentation label Aug 30, 2020
@seratch seratch added this to the 0.4.0a0 milestone Aug 30, 2020
@seratch seratch self-assigned this Aug 30, 2020
from sanic.request import Request

api = Sanic()
api = Sanic(name="awesome-slack-app")

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

name=None is deprecated

@seratch seratch merged commit a4a8634 into slackapi:main Sep 1, 2020
@seratch seratch deleted the docker-samples branch September 1, 2020 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant