-
Notifications
You must be signed in to change notification settings - Fork 36
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
Rename services container throughout #474
Conversation
Though the style guide says to just use imperatives, "get started" just sounds weird. Also this is more consistent with "troubleshooting"
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.
Thanks for the quick change!
Can you ensure the steps work without any other changes with the new image?
custom_dc/build_image.md
Outdated
@@ -124,9 +124,9 @@ If you want to create a new source directory and start from scratch, clone the r | |||
Run the following command to build the repo: | |||
|
|||
<pre> | |||
docker build --tag datacommons-website-compose:<var>DOCKER_TAG</var> \ | |||
docker build --tag datacommons-services:<var>DOCKER_TAG</var> \ | |||
-f build/web_compose/Dockerfile \ |
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.
This needs to be -f build/cdc_services/Dockerfile \
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.
Done.
No description provided.