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

Docker Files Changes #5

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

Docker Files Changes #5

wants to merge 3 commits into from

Conversation

Tonnix
Copy link

@Tonnix Tonnix commented Apr 28, 2021

  1. I've added documenting comments to flint and flint base docker files
  2. I've added a sleek docker file to the mix to provide a single resource point for dockers
  3. I've changed the source of boost libraries to avoid this issue: boost_1_70_0.tar.gz is Forbidden boostorg/boost#299
  4. I've updated the references to FLINT.Data repository which was renamed to FLINT.Data_Preprocessing
  5. I've changed the image names to correspond to the new naming standards
  6. I've updated the readme to include the fact that implementation docker files are in the repository now

@HarshCasper HarshCasper requested a review from aornugent August 28, 2021 17:17
@aornugent
Copy link

aornugent commented Aug 29, 2021

Hey @Tonnix - this looks awesome!

Tagging @malfrancis @mfellows @leitchy @kaskou - can we leverage this PR to address: #24

Makefiles look like the canonical way to publish images on https://hub.docker.com/u/mojaglobal. @Tonnix - do you think you could help one of the Documentation Working Group document the process of publishing a new Docker images? Including steps to promote new releases and and encouraging project maintainers to standardise their Dockerfiles to use the official base images?

@aornugent
Copy link

Hey @Tonnix - is there any way we can help you move this along?

@Tonnix
Copy link
Author

Tonnix commented Oct 12, 2021

Hi @aornugent ,

Yes, please let me know who I can work with in the Documentation Work Group to move this along. I will also need to talk with @malfrancis or @leitchy about it

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

Successfully merging this pull request may close these issues.

2 participants