Merge pull request #1962 from technosophos/fix/2.2.0-style-fixes

fix(style): add missing comments
pull/1360/merge
Matt Butcher 8 years ago committed by GitHub
commit ee2aaff7a6

@ -316,6 +316,7 @@ func UpgradeDryRun(dry bool) UpdateOption {
}
}
// ResetValues will (if true) trigger resetting the values to their original state.
func ResetValues(reset bool) UpdateOption {
return func(opts *options) {
opts.resetValues = reset

@ -28,6 +28,7 @@ import (
"k8s.io/helm/pkg/kube"
)
// New creates a new and initialized tunnel.
func New(namespace string, client *internalclientset.Clientset, config *restclient.Config) (*kube.Tunnel, error) {
podName, err := getTillerPodName(client.Core(), namespace)
if err != nil {

@ -39,6 +39,7 @@ const (
ReleaseTestFailure = "test-failure"
)
// FilterTestHooks filters the list of hooks are returns only testing hooks.
func FilterTestHooks(hooks []*release.Hook) ([]*release.Hook, error) {
testHooks := []*release.Hook{}
notFoundErr := errors.New("no tests found")

Loading…
Cancel
Save