Skip to content

Conversation

@stephen-hoover
Copy link
Contributor

@stephen-hoover stephen-hoover commented Feb 14, 2018

The DockerHub autobuild includes the README from GitHub, which is good. However, the README includes relative links which GitHub parses correctly but DockerHub does not. Therefore, direct users to read the README on GitHub instead of DockerHub.

Closes #54 .

The DockerHub autobuild includes the README from GitHub, which is good. However, the README includes relative links which GitHub parses correctly but DockerHub does not. Therefore, direct users to read the README on GitHub instead of DockerHub.
Copy link
Contributor

@mheilman mheilman left a comment

Choose a reason for hiding this comment

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

LGTM

@stephen-hoover stephen-hoover merged commit dab11a0 into civisanalytics:master Feb 14, 2018
@stephen-hoover stephen-hoover deleted the note-for-dockerhub-readers branch February 14, 2018 19:06
mheilman pushed a commit to mheilman/datascience-python that referenced this pull request Sep 30, 2020
The DockerHub autobuild includes the README from GitHub, which is good. However, the README includes relative links which GitHub parses correctly but DockerHub does not. Therefore, direct users to read the README on GitHub instead of DockerHub.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants