Skip to content

Commit

Permalink
Merge pull request #243 from apache/govet-fix
Browse files Browse the repository at this point in the history
Fixing some govet findings
  • Loading branch information
oscerd authored Nov 28, 2018
2 parents 5baf456 + af80f49 commit 09c6c23
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
3 changes: 1 addition & 2 deletions pkg/apis/camel/v1alpha1/types_support.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ func NewIntegrationContextList() IntegrationContextList {
}
}


// TraitProfileByName returns the trait profile corresponding to the given name (case insensitive)
func TraitProfileByName(name string) TraitProfile {
for _, p := range allTraitProfiles {
Expand All @@ -107,4 +106,4 @@ func TraitProfileByName(name string) TraitProfile {
}
}
return ""
}
}
2 changes: 0 additions & 2 deletions pkg/client/cmd/run.go
Original file line number Diff line number Diff line change
Expand Up @@ -394,12 +394,10 @@ func toJSON(value runtime.Object) ([]byte, error) {
u, err := k8sutil.UnstructuredFromRuntimeObject(value)
if err != nil {
return nil, fmt.Errorf("error creating unstructured data: %v", err)
return nil, err
}
data, err := runtime.Encode(unstructured.UnstructuredJSONScheme, u)
if err != nil {
return nil, fmt.Errorf("error marshalling to json: %v", err)
return nil, err
}
return data, nil
}
Expand Down
2 changes: 0 additions & 2 deletions pkg/util/watch/watch.go
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,4 @@ func HandleStateChanges(ctx context.Context, integration *v1alpha1.Integration,
}
}
}

return nil
}

0 comments on commit 09c6c23

Please sign in to comment.