Skip to content
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

Package publisher as container #97

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Package publisher as container #97

wants to merge 2 commits into from

Conversation

chgl
Copy link

@chgl chgl commented May 12, 2020

Closes #93

To make this really useful, this repository should be connected to Docker Hub to automatically build and publish the container image.

@grahamegrieve
Copy link
Contributor

@markiantorno this is for you when you rebuild the pipelines

@chgl
Copy link
Author

chgl commented Oct 31, 2020

Any update on this? Or any way I can help integrate a container build into the ci pipeline?

@grahamegrieve
Copy link
Contributor

@markiantorno is still working on dockerizing the validator. We'll work on dockerizing the publisher once that's settled

@lmckenzi lmckenzi added the NeedGrahame Need Grahame for discussion label Nov 22, 2022
@lmckenzi lmckenzi added enhancement New feature or request Approved Change has been reviewed and accepted and can now be applied to the templates and removed NeedGrahame Need Grahame for discussion labels Nov 29, 2022
@jkiddo
Copy link
Contributor

jkiddo commented Sep 29, 2023

Is this PR still a thing?

@dotasek
Copy link
Collaborator

dotasek commented Sep 29, 2023

@jkiddo not in this form.

There appears to be a current and better maintained dockerized publisher here: https://github.com/FHIR/auto-ig-builder/blob/master/images/ig-publisher-base/Dockerfile

I feel as though that should be used in place of this one, but have yet to discuss it with @jmandel

@jkiddo
Copy link
Contributor

jkiddo commented Sep 29, 2023

It says 'latest-sushi' - wouldnt it be in favour to have that narrowed down to a specific version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Change has been reviewed and accepted and can now be applied to the templates enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

Package Publisher as Docker Container
5 participants