File conv.go has 546 lines of code (exceeds 500 allowed). Consider refactoring.

Open

// Package convert provides functions for converting data and functions between Go and Starlark.
package convert

import (
    "errors"

Function toValue has 64 lines of code (exceeds 50 allowed). Consider refactoring.

Open

func toValue(val reflect.Value, tagName string) (result starlark.Value, err error) {
    defer func() {
        if r := recover(); r != nil {
            err = fmt.Errorf("panic recovered: %v", r)
        }

Avoid deeply nested control flow statements.

Open

            if unwrapped.Type().ConvertibleTo(targetType) {
                return unwrapped.Convert(targetType), nil
            } else if sv, ok := unwrapped.Interface().(starlark.Value); ok {
                // TODO: this path is not reachable in the current test, maybe we can remove it?
                goVal := FromValue(sv)

Function convertElemValue has 7 return statements (exceeds 4 allowed).

Open

func convertElemValue(val reflect.Value, targetType reflect.Type) (reflect.Value, error) {
    if val.Type().AssignableTo(targetType) || val.Type().ConvertibleTo(targetType) {
        return val.Convert(targetType), nil
    } else if val.Kind() == reflect.Ptr || val.Kind() == reflect.Interface {
        if val.IsNil() {

Function toValue has a Cognitive Complexity of 23 (exceeds 20 allowed). Consider refactoring.

Open

func toValue(val reflect.Value, tagName string) (result starlark.Value, err error) {
    defer func() {
        if r := recover(); r != nil {
            err = fmt.Errorf("panic recovered: %v", r)
        }

• Read up
  Read up

Function convertReflectValue has 6 return statements (exceeds 4 allowed).

Open

func convertReflectValue(val reflect.Value, argT reflect.Type) (reflect.Value, error) {
    if !val.IsValid() {
        return reflect.Zero(argT), nil
    }
    if val.Type().AssignableTo(argT) {

Function makeOut has 6 return statements (exceeds 4 allowed).

Open

func makeOut(out []reflect.Value, tagName string) (starlark.Value, error) {
    if len(out) == 0 {
        return starlark.None, nil
    }
    last := out[len(out)-1]

Function toValue has 5 return statements (exceeds 4 allowed).

Open

func toValue(val reflect.Value, tagName string) (result starlark.Value, err error) {
    defer func() {
        if r := recover(); r != nil {
            err = fmt.Errorf("panic recovered: %v", r)
        }