-
Notifications
You must be signed in to change notification settings - Fork 675
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
vapor 2 -> 3 #199
vapor 2 -> 3 #199
Conversation
Hi @tanner0101, Thanks for updating However you are using a branch, not a tag. I'll rather wait until |
@waghanza yes sorry should have mentioned that. The tag will be coming today, so wanted to have this PR ready 👍 |
@tanner0101 any update ? |
@waghanza just updated to 3.0.0 official release. 👍 |
@tanner0101 The Could you confirm that the commit I have made is OK ? |
swift/vapor/Dockerfile
Outdated
@@ -6,4 +6,4 @@ COPY Package.swift main.swift ./ | |||
|
|||
RUN swift build -c release | |||
|
|||
CMD .build/release/server serve -e prod -h 0.0.0.0 -p 3000 | |||
CMD .build/release/server serve --hostname 0.0.0.0 --port 3000 --env prod |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, that should have been -H
(capital). The change you made works, too. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, you could also write this as:
CMD .build/release/server serve -b 0.0.0.0:3000 -e prod
No description provided.