Skip to content

fix: use repo root as Docker build context#2

Merged
Sathvik-Rao merged 1 commit into
mainfrom
fix-docker
May 28, 2026
Merged

fix: use repo root as Docker build context#2
Sathvik-Rao merged 1 commit into
mainfrom
fix-docker

Conversation

@Sathvik-Rao
Copy link
Copy Markdown
Owner

@Sathvik-Rao Sathvik-Rao commented May 27, 2026

Summary

Add .dockerignore to keep build context small and exclude local/dev files. Update CI workflow and docker-compose to set the build context to the repository root and point to backend Dockerfiles. Adjust backend Dockerfile and Dockerfile.test to copy files from backend/ (requirements, app, tests, pytest.ini) and include frontend assets in the image. These changes let builds run from the repo root while avoiding sending unnecessary files to the Docker daemon.

Type of change

  • fix: ... — bug fix → patch bump (0.0.x)
  • feat: ... — new feature → minor bump (0.x.0)
  • feat!: ... / BREAKING CHANGE — breaking change → major bump (x.0.0)
  • chore: / docs: / ci: / refactor: — no release created

PR title format

Your PR title becomes the commit message on merge and determines the version bump:

feat(resume): add cover letter section
fix: pdf export crash on empty work history
feat!: replace /generate endpoint with /build (breaking)
chore: update dev dependencies

Pattern: type(optional-scope): short description in lowercase

Checklist

  • Tests added / updated under backend/tests/ for any changed backend/app/ code
  • pytest passes locally (cd backend && pytest)
  • PR title follows the format above

Add .dockerignore to keep build context small and exclude local/dev files. Update CI workflow and docker-compose to set the build context to the repository root and point to backend Dockerfiles. Adjust backend Dockerfile and Dockerfile.test to copy files from backend/ (requirements, app, tests, pytest.ini) and include frontend assets in the image. These changes let builds run from the repo root while avoiding sending unnecessary files to the Docker daemon.
@github-actions
Copy link
Copy Markdown

Suggested tests for this PR

PR title: fix: use repo root as Docker build context

PR description:

Summary

Add .dockerignore to keep build context small and exclude local/dev files. Update CI workflow and docker-compose to set the build context to the repository root and point to backend Dockerfiles. Adjust backend Dockerfile and Dockerfile.test to copy files from backend/ (requirements, app, tests, pytest.ini) and include frontend assets in the image. These changes let builds run from the repo root while avoiding sending unnecessary files to the Docker daemon.

Type of change

  • fix: ... — bug fix → patch bump (0.0.x)
  • feat: ... — new feature → minor bump (0.x.0)
  • feat!: ... / BREAKING CHANGE — breaking change → major bump (x.0.0)
  • chore: / docs: / ci: / refactor: — no release created

PR title format

Your PR title becomes the commit message on merge and determines the version bump:

feat(resume): add cover letter section
fix: pdf export crash on empty work history
feat!: replace /generate endpoint with /build (breaking)
chore: update dev dependencies

Pattern: type(optional-scope): short description in lowercase

Checklist

  • Tests added / updated under backend/tests/ for any changed backend/app/ code
  • pytest passes locally (cd backend && pytest)
  • PR title follows the format above

No backend/app/**.py files changed — no test suggestions.

@Sathvik-Rao Sathvik-Rao merged commit 5d276a4 into main May 28, 2026
4 checks passed
@Sathvik-Rao Sathvik-Rao deleted the fix-docker branch May 28, 2026 05: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