Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

feat: add version cmd #280

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

feat: add version cmd #280

wants to merge 20 commits into from

Conversation

tedim52
Copy link
Contributor

@tedim52 tedim52 commented Oct 15, 2024

Addresses #275

@tedim52 tedim52 requested a review from leoporoli October 15, 2024 16:18
@@ -0,0 +1,54 @@
#!/usr/bin/env bash
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this file? because I see there is a nix version

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we actually don't im gonna remove those

Copy link
Contributor

Choose a reason for hiding this comment

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

get-docker-tag.sh is called by generate_kardinal_version.nix I wonder is there is a way to replace it with the nix version of this script?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ye i'm actually gonna embed the code for that directly in the generate nix

Copy link
Contributor

@leoporoli leoporoli left a comment

Choose a reason for hiding this comment

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

The code is ok but it won't resolve the issue until the version generation gets connected to the release process

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

Successfully merging this pull request may close these issues.

2 participants