Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
This moves
main.go
to a directory that matches the name of our installed binary, as recommended by the cobra docs:https://github.com/spf13/cobra/blob/7547e83b2d85fd1893c7d76916f67689d761fecb/README.md#getting-started
This will make it possible to install
ct
with the commandgo get github.com/helm/chart-testing/ct
if merged to master.Which issue this PR fixes
Closes #109
Special notes for your reviewer:
Since
go get
doesn't allow specifying a branch, for testing purposes I modified imports on the master branch of my fork. You can rungo get github.com/jlegrone/chart-testing/ct
to test go get behavior.