Skip to content
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

Move to Docker instead of Azure Functions #22

Merged
merged 11 commits into from
Nov 16, 2024
Merged

Conversation

tblanarik
Copy link
Owner

This pull request includes significant changes to transition the Spotbot application from an Azure Function App to a Docker container deployed on Azure Web App. The most important changes include the addition of a Dockerfile, a new GitHub Actions workflow for deployment, and updates to the application code to support Flask instead of Azure Functions.

Choose a reason for hiding this comment

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

Copilot reviewed 6 out of 10 changed files in this pull request and generated no suggestions.

Files not reviewed (4)
  • .github/workflows/main_hamalertspotbot.yml: Language not supported
  • Dockerfile: Language not supported
  • host.json: Language not supported
  • cleanup.py: Evaluated as low risk
Comments skipped due to low confidence (1)

gunicorn_config.py:5

  • [nitpick] Consider removing the commented-out line or adding a comment explaining why it's commented out.
# timeout = int(os.environ.get('GUNICORN_TIMEOUT', '120'))

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@tblanarik tblanarik merged commit 0795c44 into main Nov 16, 2024
1 check passed
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.

1 participant