Skip to content

Merge Feature/add dockerfile#247

Closed
rugitter wants to merge 1 commit intoDispatch-AI-com:mainfrom
Dispatch-AI-2026:main
Closed

Merge Feature/add dockerfile#247
rugitter wants to merge 1 commit intoDispatch-AI-com:mainfrom
Dispatch-AI-2026:main

Conversation

@rugitter
Copy link
Copy Markdown

  • Added new docker file
  1. optimise pnpm install process
  2. add docker-compose file
  3. fix EOL error when build cross platform
  4. ignore eslint during build to prevent build failure due to lint error
  • Improve Dockerfile
  1. Add appuser to run the app
  2. Optimise runtime image in speed and size

* Added new docker file
1. optimise pnpm install process
2. add docker-compose file
3. fix EOL error when build cross platform
4. ignore eslint during build to prevent build failure due to lint error

* Improve Dockerfile
1. Add appuser to run the app
2. Optimise runtime image in speed and size

---------

Co-authored-by: Charles Cheng <csuchq@gmail.com>
Copilot AI review requested due to automatic review settings March 15, 2026 09:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds containerization support for the Next.js frontend by introducing a production Dockerfile + docker-compose workflow, and adjusts Next.js build output to support a standalone runtime image.

Changes:

  • Configure Next.js for output: 'standalone' and skip ESLint during next build.
  • Add a new multi-stage production Dockerfile and a docker-compose.yml service.
  • Add .gitattributes for LF line endings and update .dockerignore to exclude .next.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
next.config.ts Enables standalone output and disables ESLint during builds.
docker-compose.yml Defines a frontend service building the Docker image and exposing port 3000.
Dockerfile Adds a multi-stage build producing a smaller non-root runtime image using Next standalone output.
.gitattributes Enforces LF EOL normalization across platforms.
.dockerignore Updates ignored build-context files, including .next/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +4 to +9
# Install pnpm
RUN corepack enable

# Copy and Install dependencies
COPY package.json pnpm-lock.yaml ./
RUN pnpm install --frozen-lockfile
styledComponents: true,
},
eslint: {
ignoreDuringBuilds: true,

node_modules/
# .next/
.next/
Comment on lines +3 to +10
build:
context: .
dockerfile: Dockerfile
ports:
- "3000:3000"
volumes:
- .:/app
- /app/node_modules
@abbyli-88 abbyli-88 closed this Mar 15, 2026
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.

3 participants