Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update with-docker example and deployment docs. #23486

Merged
merged 2 commits into from Mar 28, 2021
Merged

Conversation

@leerob
Copy link
Contributor

@leerob leerob commented Mar 28, 2021

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes #number
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.

Documentation / Examples

  • Make sure the linting passes
@@ -124,7 +132,7 @@ EXPOSE 3000
# Uncomment the following line in case you want to disable telemetry.
# RUN npx next telemetry disable
CMD ["node_modules/.bin/next", "start"]
CMD ["yarn", "start"]

This comment has been minimized.

@leerob

leerob Mar 28, 2021
Author Contributor

I changed this because Google Cloud Run requires you to forward a port to the start script. Then I realized it's probably confusing if package.json isn't the source of truth for how you're starting your application, so it's now looking there.


You can view your images created with `docker images`.

## Deploying to Google Cloud Run

This comment has been minimized.

@leerob

leerob Mar 28, 2021
Author Contributor

Confirmed this works 👍

@ijjk
ijjk approved these changes Mar 28, 2021
@kodiakhq kodiakhq bot merged commit d49f978 into vercel:canary Mar 28, 2021
23 checks passed
23 checks passed
build
Details
Cancel Previous Runs
Details
PR Stats
Details
testYarnPnP
Details
Webpack 5 (Basic, Production, Acceptance)
Details
React 16 + Webpack 4 (Basic, Production, Acceptance)
Details
lint
Details
Check Pre-compiled
Details
Test Unit
Details
Test Integration (1)
Details
Test Integration (2)
Details
Test Integration (3)
Details
Test Integration (4)
Details
Test Integration (5)
Details
Test Integration (6)
Details
Test Electron
Details
Test Firefox (production)
Details
Test Safari (production)
Details
Potentially publish release
Details
thank you, next
Details
Test Safari 10.1 (nav)
Details
Release Stats
Details
kodiakhq: status ⛴ attempting to merge PR (merging)
Details
@leerob leerob deleted the leerob:docker-docs branch Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants