Add missing mandatory arguments.#1986
Conversation
The command given herein is incomplete, and it will lead to an erroneous outcome, which discourages the learners badly.
|
@londoncalling PTAL.I think if your compose file is |
|
@JerryChin , @mstanleyjones the command is correct, as written in the appropriate section of the tutorial. The page you updated to include The steps in the next topics (after you've done the set up and gotten the swarm running) show the full format of the command: Deploy the app
(we use the long form, @ManoMarks any other thoughts or do you want to jump in? |
|
@JerryChin I'm going to close this PR when PR #2042 is merged. That should address your feedback more thoroughly and in line with the way the tutorial is written. Thanks! |
|
Okay, let's do it!
|
|
@JerryChin Cool, thanks. The updates from PR #2042 are published. The topics I updated per your feedback are: I'm closing this PR. Please send more comments if needed. |
Proposed changes
Reason
The command given herein is incomplete, and it will lead to an erroneous outcome, which discourages the learners badly.
Action
I completed the command with the proper option and argument.
Unreleased project version (optional)
N/A
Related issues (optional)
N/A