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

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 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