Skip to content

Deployment-related workflow improvements#54

Open
onyxfish wants to merge 4 commits intomainfrom
feat/workflow-improvements
Open

Deployment-related workflow improvements#54
onyxfish wants to merge 4 commits intomainfrom
feat/workflow-improvements

Conversation

@onyxfish
Copy link
Contributor

@onyxfish onyxfish commented Mar 12, 2026

Apologies for a grab bag, but this PR is a bunch fo small deployment-adjacent workflow improvements

  • Added explicit documentation that this repository uses Github flow
  • Added command to Dockerfile ensure apt dependencies are updated in the container at build time
  • Added healthchecks to the Docker image and the ECS configuration (sharing a script)
  • Configured CI to run on PR rather than push (to gate merges)
  • Removed unused arguments from the CI configuration
  • Added precommit hook for ruff and vulture

I also changed the following GitHub settings:

  • Protected main branch
  • Required 1 approval for merge
  • Required squash merges for clean git history

Up next:

@onyxfish onyxfish marked this pull request as ready for review March 12, 2026 22:14
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