Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

chore: Add metadata to Dockerfile #21

Merged
merged 1 commit into from
Dec 17, 2019
Merged

chore: Add metadata to Dockerfile #21

merged 1 commit into from
Dec 17, 2019

Conversation

RichardLitt
Copy link
Contributor

Adding the usual suspects. This seems to be the format, according to this page.

Adding the usual suspects. This seems to be the format, according to [this page](https://docs.docker.com/engine/reference/builder/).
@aphelionz
Copy link
Contributor

@RichardLitt Looking good. Do we have this on docker hub? We'd have to take on the burden of maintaining it (i.e. adding it to the release checklist) but it does seem like this, in particular, would be valuable to have as a reliable container to run

@RichardLitt
Copy link
Contributor Author

We do. It should be part of the release process. I'm not sure if we have that documented, anywhere, though. @phillmac Might know?

@phillmac phillmac merged commit 8eaa3a7 into master Dec 17, 2019
@phillmac
Copy link
Contributor

phillmac commented Dec 17, 2019

That's probably something I should get fixed. One of the main issues is getting a stable enough js-ipfs version to include in the imag; the dht seems to cause endless problems; so I've held off on including all the hackery I've been trying to get working and this repo has stagnated a little as a result.

@phillmac phillmac deleted the feat/dockerfile branch December 17, 2019 06:11
@RichardLitt
Copy link
Contributor Author

Fair enough. Something for the future. Cross ref #22.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants