Skip to content

.github/workflows: add containers.yaml#46

Merged
mostlygeek merged 1 commit into
mainfrom
mostlygeek/containers
Sep 18, 2025
Merged

.github/workflows: add containers.yaml#46
mostlygeek merged 1 commit into
mainfrom
mostlygeek/containers

Conversation

@mostlygeek
Copy link
Copy Markdown
Contributor

Add a GH action to build and publish a docker container when a new tag is pushed.

Updates #41

@mostlygeek mostlygeek self-assigned this Sep 17, 2025
@mostlygeek mostlygeek force-pushed the mostlygeek/containers branch from c360488 to 47d54d2 Compare September 18, 2025 00:54
@mostlygeek mostlygeek requested a review from awly September 18, 2025 00:55
Comment thread .github/workflows/containers.yaml Outdated
Comment thread .github/workflows/containers.yaml
Comment thread .github/workflows/containers.yaml Outdated
Comment thread .github/workflows/containers.yaml Outdated
Comment thread .github/workflows/containers.yaml Outdated
if [[ "$REF" == refs/tags/* ]]; then
TAG="${REF#refs/tags/}"
else
TAG="latest"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this will end up outputting the latest tag twice.
maybe just echo the tags here and above, and exit?

Comment thread .github/workflows/containers.yaml Outdated
Comment thread .github/workflows/containers.yaml Outdated
@mostlygeek mostlygeek force-pushed the mostlygeek/containers branch from 0c6b1e2 to c3d70df Compare September 18, 2025 03:48
Add a GH action to build and publish a docker container when a new tag
is pushed.

Updates #41

Signed-off-by: Benson Wong <benson@tailscale.com>
@mostlygeek mostlygeek force-pushed the mostlygeek/containers branch from c3d70df to 496c292 Compare September 18, 2025 03:53
@mostlygeek
Copy link
Copy Markdown
Contributor Author

@awly thanks for the earlier review. Incorporated your feedback please take another look.

@mostlygeek mostlygeek merged commit 21038be into main Sep 18, 2025
3 checks passed
@mostlygeek mostlygeek deleted the mostlygeek/containers branch September 18, 2025 17:04
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