Skip to content

Add task to build and publish docker images #341

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

Merged
merged 1 commit into from
Aug 8, 2023
Merged

Conversation

lkitching
Copy link
Contributor

Issue #340 - Add docker task to build.clj which allows docker images to be built locally, and optionally published to the public swirrl artifact repository.

Add deploy-docker task to the CI build to publish docker images.

Update the README to include instructions on running csv2rdf via docker.

Issue #340 - Add docker task to build.clj which allows docker images
to be built locally, and optionally published to the public swirrl
artifact repository.

Add deploy-docker task to the CI build to publish docker images.

Update the README to include instructions on running csv2rdf via
docker.
@lkitching lkitching requested a review from scottlowe August 7, 2023 11:15
@@ -124,6 +124,18 @@ jobs:
"${CIRCLE_TAG}" \
csv2rdf-darwin.tar.gz

deploy-docker:
docker:
- image: cimg/clojure:1.11.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I personally prefer locking the image to a known JDK version which we intend to support e.g. this snippet from the muttnik project:

Screenshot 2023-08-08 at 3 16 47 PM

@scottlowe scottlowe merged commit 00626ce into master Aug 8, 2023
@scottlowe scottlowe deleted the lk/issue_340 branch August 8, 2023 19:30
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