Skip to content

fix: docker file and deployment#60

Merged
shabebe11 merged 2 commits into
mainfrom
deployment-fix
May 13, 2026
Merged

fix: docker file and deployment#60
shabebe11 merged 2 commits into
mainfrom
deployment-fix

Conversation

@shabebe11
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 13, 2026 21:44
Copy link
Copy Markdown

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

This PR updates the project’s containerized deployment path for a Next.js/Bun app on Fly.io.

Changes:

  • Replaces the static-file Docker image with a Bun-based multi-stage Next.js Docker build.
  • Aligns Fly.io service port with the app’s Next.js runtime port.
  • Updates build scripts and generated Next type references for the deployment workflow.

Reviewed changes

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

Show a summary per file
File Description
package.json Switches build-related scripts from Node to Bun execution.
next-env.d.ts Updates the generated route type import path.
fly.toml Changes Fly.io internal service port to 3000.
Dockerfile Adds a Bun-based build/runtime image for the Next.js app.
.dockerignore Expands Docker build-context exclusions for app deployment.

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

Comment thread Dockerfile Outdated
Comment thread Dockerfile
Comment thread package.json Outdated
@shabebe11 shabebe11 merged commit 9a45b8d into main May 13, 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.

2 participants