Showing 37 of 37 total issues
Function hasMapListAMatchingItem
has a Cognitive Complexity of 25 (exceeds 20 allowed). Consider refactoring. Open
Open
func hasMapListAMatchingItem(searchFor map[string]interface{}, typedMapListItem map[string]interface{}) bool {
klog.V(5).InfoS("hasMapListAMatchingItem will search", "searchFor", searchFor, "typedMapListItem", typedMapListItem)
if len(searchFor) == 0 {
return false
}
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function isStatusConditionHealthy
has 51 lines of code (exceeds 50 allowed). Consider refactoring. Open
Open
func isStatusConditionHealthy(condition map[string]interface{}) bool {
switch {
/*
From https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#typical-status-properties:
Function hasMapListAMatchingItem
has 9 return statements (exceeds 4 allowed). Open
Open
func hasMapListAMatchingItem(searchFor map[string]interface{}, typedMapListItem map[string]interface{}) bool {
klog.V(5).InfoS("hasMapListAMatchingItem will search", "searchFor", searchFor, "typedMapListItem", typedMapListItem)
if len(searchFor) == 0 {
return false
}
Function Run
has 6 return statements (exceeds 4 allowed). Open
Open
func Run(f util.Factory, streams genericiooptions.IOStreams, args []string) error {
klog.V(5).InfoS("All config settings", "settings", viper.AllSettings())
if viper.Get("color") == "always" {
color.NoColor = false
} else if viper.Get("color") == "never" {
Function runWatch
has 6 return statements (exceeds 4 allowed). Open
Open
func runWatch(results *resource.Result, engine *renderEngine) error {
color.HiYellow("\nPrinted all existing resource statuses, starting to watch. Switching to shallow mode during watch!\n\n")
viper.Set("shallow", true)
viper.Set("watching", true)
klog.V(5).InfoS("Will run watch")
Method renderEngine.mappingFor
has 6 return statements (exceeds 4 allowed). Open
Open
func (e *renderEngine) mappingFor(resourceOrKindArg string) (*meta.RESTMapping, error) {
fullySpecifiedGVR, groupResource := schema.ParseResourceArg(resourceOrKindArg)
gvk := schema.GroupVersionKind{}
restMapper, err := e.repo.ToRESTMapper()
if err != nil {
Method RenderableObject.kubeGetUnifiedDiffString
has 5 return statements (exceeds 4 allowed). Open
Open
func (r RenderableObject) kubeGetUnifiedDiffString(resourceOrKind, namespace, nameA, nameB string) (string, error) {
controllerRevisionMapping, err := r.engine.mappingFor(resourceOrKind)
if err != nil {
klog.V(3).ErrorS(err, "failed to get mapping", "resourceOrKind", resourceOrKind)
return "", err
should replace count += 1 with count++ Open
Open
count += 1
- Exclude checks
exported method RenderableObject.KubeGetIngressesMatchingService should have comment or be unexported Open
Open
func (r RenderableObject) KubeGetIngressesMatchingService(namespace, svcName string) (out []RenderableObject) {
- Exclude checks
exported method RenderableObject.IncludeRenderableObject should have comment or be unexported Open
Open
func (r RenderableObject) IncludeRenderableObject(obj RenderableObject) (output string) {
- Exclude checks
exported method RenderableObject.Metadata should have comment or be unexported Open
Open
func (r RenderableObject) Metadata() (metadata map[string]interface{}) {
- Exclude checks
if block ends with a return statement, so drop this else and outdent its block Open
Open
} else {
- Exclude checks
exported method RenderableObject.KubeGet should have comment or be unexported Open
Open
func (r RenderableObject) KubeGet(namespace string, args ...string) (out []RenderableObject) {
- Exclude checks
exported method RenderableObject.KubeGetNodeStatsSummary should have comment or be unexported Open
Open
func (r RenderableObject) KubeGetNodeStatsSummary(nodeName string) map[string]interface{} {
- Exclude checks
exported method RenderableObject.Kind should have comment or be unexported Open
Open
func (r RenderableObject) Kind() (kind string) {
- Exclude checks
exported method RenderableObject.Labels should have comment or be unexported Open
Open
func (r RenderableObject) Labels() (labels map[string]interface{}) {
- Exclude checks
2: cannot find package "github.com/bergerx/kubectl-status/pkg/plugin" in any of: Open
Open
"github.com/bergerx/kubectl-status/pkg/plugin"
- Exclude checks
exported method ResourceRepo.ToRESTMapper should have comment or be unexported Open
Open
func (r *ResourceRepo) ToRESTMapper() (meta.RESTMapper, error) {
- Exclude checks
exported method RenderableObject.Annotations should have comment or be unexported Open
Open
func (r RenderableObject) Annotations() (annotations map[string]interface{}) {
- Exclude checks
exported method ResourceRepo.CLIQueryResults should have comment or be unexported Open
Open
func (r *ResourceRepo) CLIQueryResults(args []string) *resource.Result {
- Exclude checks