Yuval Goldberg

Results 21 comments of Yuval Goldberg

This PR resolves https://github.com/alexellis/arkade/pull/687 and future bumps of Hashicorp tools (except for Waypoint that isn't yet available on https://checkpoint-api.hashicorp.com/ )

What do you think about changing the `Version` field from a `string` to a function returns a `string`? Then, each tool can be implemented according to its requirements. https://github.com/alexellis/arkade/blob/19077193030259be439587b2686b0d4530ced248/pkg/get/get.go#L35-L37 Current...

> Hi @YuviGold > > Changing Version from a string to function would involve a massive refactor. I think we could probably add a separate field with a different name...

Just checked about the different URLs Click to expand the code: ```go func Test_Tools(t *testing.T) { tools := MakeTools() nonGitHubURLs := make([]string, 0) for _, tool := range tools {...

> Hey @YuviGold that's an interesting feature to have in the app. But we are not sure if we want to download Docker from `arkade` itself. Both Mac and Windows...

Thank you @alexellis I created 2 PRs: - [x] https://github.com/alexellis/arkade/pull/529 - [ ] https://github.com/spf13/cobra/pull/1500 I hope cobra will look into it soon :)

Closed the PR https://github.com/alexellis/arkade/pull/737 till deciding on the technical solution. The solution needs to support: - Generation script needs to run with the latest binary (in order to get latest...

Hi @johnSchnake I already implemented a solution on https://github.com/spf13/cobra/pull/1500 but the maintainers of the repository had mixed thoughts about it and didn't get to a consensus how to address this...