Preparing report...

Report for github.com/tinynetwork/tinet

B    Not bad!    Found 5 issues across 7 files

Tweet

gofmt71%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


gocyclo71%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.


golint42%

Golint is a linter for Go source code.

    • tinet/command_func.go
    • Line 27: warning: exported function LoadCfg should have comment or be unexported (golint)
    • Line 50: warning: exported function CmdBuild should have comment or be unexported (golint)
    • Line 66: warning: exported function CmdCheck should have comment or be unexported (golint)
    • Line 113: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 122: warning: exported function CmdUp should have comment or be unexported (golint)
    • Line 232: warning: exported function CmdConf should have comment or be unexported (golint)
    • Line 253: warning: exported function CmdUpConf should have comment or be unexported (golint)
    • Line 267: warning: exported function CmdDown should have comment or be unexported (golint)
    • Line 286: warning: exported function CmdExec should have comment or be unexported (golint)
    • Line 301: warning: exported function CmdImg should have comment or be unexported (golint)
    • Line 349: warning: exported function CmdInit should have comment or be unexported (golint)
    • Line 361: warning: exported function CmdPs should have comment or be unexported (golint)
    • Line 378: warning: exported function CmdPull should have comment or be unexported (golint)
    • Line 390: warning: exported function CmdReConf should have comment or be unexported (golint)
    • Line 409: warning: exported function CmdReUp should have comment or be unexported (golint)
    • Line 423: warning: exported function CmdTest should have comment or be unexported (golint)
    • tinet/main.go
    • Line 16: warning: exported var Version should have comment or be unexported (golint)
    • tinet/internal/pkg/shell/shell.go
    • Line 30: warning: comment on exported type PreCmd should be of the form "PreCmd ..." (with optional leading article) (golint)
    • Line 36: warning: comment on exported type PreInit should be of the form "PreInit ..." (with optional leading article) (golint)
    • Line 41: warning: comment on exported type PostInit should be of the form "PostInit ..." (with optional leading article) (golint)
    • Line 46: warning: comment on exported type PostFini should be of the form "PostFini ..." (with optional leading article) (golint)
    • Line 51: warning: comment on exported type Node should be of the form "Node ..." (with optional leading article) (golint)
    • Line 71: warning: comment on exported type Interface should be of the form "Interface ..." (with optional leading article) (golint)
    • Line 79: warning: comment on exported type Sysctl should be of the form "Sysctl ..." (with optional leading article) (golint)
    • Line 84: warning: exported type Template should have comment or be unexported (golint)
    • Line 90: warning: comment on exported type Switch should be of the form "Switch ..." (with optional leading article) (golint)
    • Line 96: warning: comment on exported type NodeConfig should be of the form "NodeConfig ..." (with optional leading article) (golint)
    • Line 102: warning: comment on exported type Cmd should be of the form "Cmd ..." (with optional leading article) (golint)
    • Line 107: warning: comment on exported type Test should be of the form "Test ..." (with optional leading article) (golint)
    • Line 113: warning: comment on exported method Node.BuildCmd should be of the form "BuildCmd ..." (golint)
    • Line 459: warning: don't use underscores in Go names; range var dns_search should be dnsSearch (golint)
    • Line 508: warning: receiver name bridge should be consistent with previous receiver name br for Switch (golint)
    • Line 584: warning: don't use underscores in Go names; method Mount_docker_netns should be MountDockerNetns (golint)
    • Line 605: warning: comment on exported method Node.DelNsCmd should be of the form "DelNsCmd ..." (golint)
    • Line 613: warning: comment on exported method Node.MountTmpl should be of the form "MountTmpl ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!