Skip to content

[dockerhub] Added action for publishing docker image #1421

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 7 commits into from
Aug 29, 2023

Conversation

mohsenari
Copy link
Collaborator

Summary

added an action to build and publish an image containing devbox and nix installations to dockerhub

How was it tested?

Not tested yet

@mohsenari mohsenari requested review from loreto and Lagoja August 28, 2023 16:02
@@ -0,0 +1,27 @@
name: docker-image-release

on: workflow_dispatch
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can test this in this PR with

on:
    push:

And if it works, change it back to workflow dispatch

uses: docker/build-push-action@v4
with:
context: ./examples/docker/
file: ./examples/docker/Dockerfile
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if you can simply run devbox generate dockerfile command here, instead of pushing a hardcoded Dockerfile (so that it does not get out of sync)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@LucilleH the output of that command is not entirely the same as the default image I want to publish.
After publishing, I'm intending to change the output of this command to use the published image

@mohsenari mohsenari merged commit f4fcd82 into main Aug 29, 2023
@mohsenari mohsenari deleted the mohsen--devbox-default-image branch August 29, 2023 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants