docker-builds icon indicating copy to clipboard operation
docker-builds copied to clipboard

Add READMEs for each tool

Open erinyoung opened this issue 3 years ago • 4 comments

This should actually be pretty straight forward.

Just a small little readme for each tool.

Then update dockerhub to have the readme for the tool instead of the main repo to make it look nice.

erinyoung avatar Feb 17 '22 21:02 erinyoung

I like this idea. What about a small template README that could be used as starting point?

k-florek avatar Feb 24 '22 17:02 k-florek

Agreed, I like the idea too.

Just not sure how to replace the current READMEs on all the dockerhub repos since they all display the main /README.md instead of tool specific READMEs.

Once a dockerhub repo is linked to GitHub and an image is built using the dockerhub auto-build feature, then the README is displayed on dockerhub. Just need to see if we can point dockerhub to tool-specific READMEs

kapsakcj avatar Feb 24 '22 17:02 kapsakcj

I just found some documentation on getting a specific readme uploaded into dockerhub for a container at https://github.com/docker/build-push-action/blob/master/docs/advanced/dockerhub-desc.md and https://github.com/peter-evans/dockerhub-description:

Specifically,

      -
        name: Update repo description
        uses: peter-evans/dockerhub-description@v2
        with:
          username: ${{ secrets.DOCKERHUB_USERNAME }}
          password: ${{ secrets.DOCKERHUB_PASSWORD }}
          repository: user/app
          readme-filepath: ./path/to/README.md

I'm going to be testing this today/soon to see if it works

erinyoung avatar Mar 25 '22 17:03 erinyoung

Turns out the 'peter-evans/dockerhub-description' route WOULD work, but doesn't because of access token issues : https://github.com/peter-evans/dockerhub-description/issues/10

There doesn't seem to be any other good options to do this, so I'm going to pause this idea for a moment and just strongly suggest that each tool has its own readme in hopes that something will appear in the future.

erinyoung avatar Apr 08 '22 18:04 erinyoung

I'm closing this issue, because it doesn't seem to be an issue anymore.

erinyoung avatar Oct 07 '22 19:10 erinyoung