-
Notifications
You must be signed in to change notification settings - Fork 5
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: --chart flag #121
feat: --chart flag #121
Conversation
internal/cmd/local/local/install.go
Outdated
@@ -42,6 +42,7 @@ const ( | |||
) | |||
|
|||
type InstallOpts struct { | |||
HelmChartFlag string |
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.
Something looks weird with the formatting here, can you verify that this was formatted?
internal/cmd/local/local_install.go
Outdated
@@ -101,6 +102,7 @@ func (i *InstallCmd) Run(ctx context.Context, provider k8s.Provider, telClient t | |||
} | |||
|
|||
opts := local.InstallOpts{ | |||
HelmChartFlag: i.Chart, |
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.
formatting here as well
Also can you update the |
I think the XOR is broken:
|
Thanks @stephanbertl . Hopefully #130 will address that. |
--chart
allows using a specific chart, such as a local chart directory (--chart /Users/alexbuchanan/dev/dev-chart
)I've been hacking a local chart path into my abctl code to do development with a local chart directory.