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
Some refactoring #9
Conversation
I suggest we change the README in a separate PR to keep the changes more isolated. Here's a small PR for that: #10 |
@@ -0,0 +1,10 @@ | |||
# _docker-library | |||
|
|||
This directory is a placeholder with samples for the following repos: |
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.
I'm a bit confused, I thought we talked about not having samples here?
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.
These are not examples for how to use our repo. These are placeholder samples of what we need to submit to the other repos as part of the process of publishing our official image.
I don't have all of the context, but this generally seems like we're moving the ball forward. |
This is just a bit of refactoring/restructuring to make the things we still need to do a bit clearer and to iterate somewhat on the README content. The content under
_docker-library
is ultimately intended to be removed from this repo and become pull requests to two other repos:We could work directly with clones of those, but it might be easier to have all the content in this repo while we iterate on it.
Also note that I followed the pattern used by the official image for Node.js (which has
docker-library/docs/node/README.me
point tonodejs/docker-node/README.md
for how to use the image, rather than the other way around like the Go team does; since we're following the Node model a bit more by maintaining our own official repo on dart-lang outside of the docker-library org, this probably makes more sense for us.