Skip to content

Conversation

JakobKlotz
Copy link
Contributor

What?

Expanded the with-docker example for usage with bun and added build instructions to the README.

Why?

The existing example supports npm, yarn and pnpm, but not bun.

How?

  • Add a separate Dockerfile.bun placed next to the existing Dockerfile, which is more sensible for the existing solution.
  • Dockerfile.bun uses the official oven/bun base image (different runtime/tooling than node:18-alpine), so a separate file should be clearer.
  • Existing Docker commands in the README remain unchanged, because Docker still uses the existing Dockerfile by default.
  • The bun example can be built with: docker build -f Dockerfile.bun -t nextjs-docker .

Related to #78465, #82998

@ijjk ijjk added the examples Issue was opened via the examples template. label Aug 26, 2025
@ijjk
Copy link
Member

ijjk commented Aug 26, 2025

Allow CI Workflow Run

  • approve CI run for commit: 6cae383

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@icyJoseph
Copy link
Contributor

Hi, could you try to address the bot comments?

@JakobKlotz
Copy link
Contributor Author

Hi, could you try to address the bot comments?

Sure, addressed them and left some comments for clarification!

Existing docker example within the same directory does not create a next.config.js for standalone output (within the Dockerfile), hence it's now more consistent.

Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
@icyJoseph icyJoseph enabled auto-merge (squash) September 3, 2025 21:54
@icyJoseph icyJoseph merged commit f41c809 into vercel:canary Sep 3, 2025
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Issue was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants