feat: add deployment for bookkeeper app#155
Conversation
|
can l please have a review thank you |
|
@leecurtis269-maker Could you review this PR when you have a moment? |
|
@leecurtis269-maker Reminder: PR #155 is ready for review. Once approvals and checks pass, you can enable auto-merge from the GitHub UI if needed. |
|
PR Summary: This change adds deployment support for the bookkeeper sample app. It includes a Dockerfile, docker-compose, Kubernetes manifests, local deployment docs, and CI workflows. PRs now run tests plus a preview Docker build workflow; main pushes run build/push and optional deploy when KUBE_CONFIG is set. Please review the workflow and merge once checks pass. |
leecurtis269-maker
changed the title
|
Updated the PR title to for clarity. I attempted to add a label as well, but the repo permissions do not allow creating labels from this integration. |
1 participant
Adds Dockerfile, docker-compose, Kubernetes manifests, and CI workflow to build/push/deploy the bookkeeper sample app. PRs now run tests and a dedicated preview Docker build workflow, while main pushes run the build/push and deploy jobs. Updated to avoid secret evaluation in PR deploy path so forked PR workflows no longer fail. Includes local deployment instructions and verified pytest coverage.