Commit Graph

9 Commits

Author SHA1 Message Date
Stefan Scherer
12a64fad04
Merge pull request #54 from klintzz/patch-1
Dockerfile CMD path fix
2020-07-08 18:11:35 +02:00
Kevin Lin
631465eee2
Removed leading slash
Minor fix
2020-07-08 08:52:39 -07:00
Kevin Lin
67a914066f
CMD update 2020-06-29 09:37:25 -07:00
Kevin Lin
28b0c3ccad
Additional CMD updates for dockerfiles 2020-06-29 09:30:32 -07:00
Stefan Scherer
c9ac9f27df
Merge pull request #50 from erichays/best-practices-fix
Match output to input and clarify build command
2020-06-29 10:54:46 +02:00
Eric Hays
1c571de3d5 Match output to input and clarify build command
The Dockerfile listed above specifically uses package.json, but the
output listed for the build lists package*, which is inconsistent. The
React example later in the doc uses package*, which would be consistent
with the output, but this should fix the mismatch for the beginner going
through the tutorial for the first time.

Additionally, the sentence "Build the Docker image now using 'docker
build' again" might be confusing to a beginner. I specifically tested
running this exact command and it gives an error message. Specifically
using the exact command needed to build this target shouldn't take up
too much space.
2020-06-03 02:10:30 +00:00
Dominik Mahnkopf
fc9025459a Add .dockerignore for node_modules & explanation 2020-06-02 22:07:07 +02:00
Rahul Kumaresan
98a838c3c5 correct intent in 'Layer Caching' description 2020-04-19 13:52:03 +05:30
Michael Irwin
5bb26cc53a
Initial commit
Signed-off-by: mikesir87 <mikesir87@gmail.com>

Imported from dockersamples/101-tutorial, removed other languages
for now, and replaced PWD references with Docker Desktop.
2020-02-05 22:04:43 -05:00