Skip to content

chore(web): ignore host build output in the web image build - #14

Merged
ayushkumar320 merged 1 commit into
iprashantraj:mainfrom
ayushkumar320:chore/web-dockerignore
Aug 22, 2026
Merged

ayushkumar320 merged 1 commit into
iprashantraj:mainfrom
ayushkumar320:chore/web-dockerignore

Conversation

@ayushkumar320

@ayushkumar320 ayushkumar320 commented Aug 22, 2026 •

Copy link
Copy Markdown
Collaborator

The web Dockerfile installs dependencies in a separate stage and copies them forward, but the following COPY . . pulled the host's node_modules and .next over them. Ignore those (plus test artifacts) so the image is built from the container's own install.

What this changes

Why

Checks

  • make ci passes (ruff, mypy --strict, pytest ≥ 80% coverage)
  • Web checks pass if apps/web changed (npm run typecheck, test:store, test:e2e)
  • Docs updated if behaviour, architecture or setup changed
  • Self-reviewed against CLAUDE.md §3 — grounded claims, no stat dumps, state discipline

The web Dockerfile installs dependencies in a separate stage and copies
them forward, but the following `COPY . .` pulled the host's node_modules
and .next over them. Ignore those (plus test artifacts) so the image is
built from the container's own install.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 22, 2026 02:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ayushkumar320
ayushkumar320 merged commit faab04f into iprashantraj:main Aug 22, 2026
2 checks passed
@ayushkumar320
ayushkumar320 deleted the chore/web-dockerignore branch August 22, 2026 02:15
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.

2 participants