-
Notifications
You must be signed in to change notification settings - Fork 50
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
Update instructions #45
Conversation
install
Outdated
@@ -30,7 +30,7 @@ fi | |||
mkdir "$BASEDIR/tanka" && cd "$BASEDIR/tanka" | |||
|
|||
# Initialise Tanka | |||
tk init --k8s=1.22 | |||
tk init --k8s=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason I changed this is that the old command used to throw this error:
Error: invalid argument "1.22" for "--k8s" flag: strconv.ParseBool: parsing "1.22": invalid syntax
Usage:
tk init [flags]
Flags:
-f, --force ignore the working directory not being empty
-h, --help help for init
--k8s set to false to skip installation of k.libsonnet (default true)
Global Flags:
-v, --verbose
Ouch: invalid argument "1.22" for "--k8s" flag: strconv.ParseBool: parsing "1.22": invalid syntax
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What version of Tanka are you using? It seems like the latest version supports version numbers: https://github.com/grafana/tanka/blob/cc213234f1e103764095e99a3ea974ca71c037cc/cmd/tk/init.go#L57-L65
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, thank you for pointing this out. I'd installed an old version because the link in the description pointed to an old one. I updated my tanka and also updated the link in the instructions. The command works now as it is!
8f72a97
to
20381cf
Compare
Minor fixes to the instructions and the installation script.