Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

CLEANUP: Tweak Dockerfile a little and add comments to make it clearer. #1569

Merged
merged 1 commit into from
May 14, 2019
Merged

CLEANUP: Tweak Dockerfile a little and add comments to make it clearer. #1569

merged 1 commit into from
May 14, 2019

Conversation

tyler-smith
Copy link
Member

Addressing #1374

@coveralls
Copy link

Coverage Status

Coverage remained the same at 35.507% when pulling 87e2bc0 on TS_docker_docs into c57238b on master.

placer14
placer14 approved these changes May 14, 2019
View reviewed changes
Copy link
Member

@placer14 placer14 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GTM. (Thanks for documenting.)

@placer14
Copy link
Member

Tested docker build locally with success. 🤝

@placer14 placer14 merged commit d3bd454 into master May 14, 2019
@placer14 placer14 deleted the TS_docker_docs branch May 14, 2019 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@placer14 placer14 placer14 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tyler-smith @coveralls @placer14