You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
⚠ misspell failed (.)
util/provider/dockermachine.go:93:26: "exportfs" is a misspelling of "exports"
util/provider/dockermachine.go:94:40: "exportfs" is a misspelling of "exports"
util/provider/dockermachine.go:95:40: "exportfs" is a misspelling of "exports"
util/provider/share/share_linux.go:296:21: "exportfs" is a misspelling of "exports"
The reason will be displayed to describe this comment to others. Learn more.
⚠ ineffassign failed (.)
/github/workspace/models/config.go:104:2: ineffectual assignment to c
/github/workspace/models/console.go:30:2: ineffectual assignment to c
/github/workspace/models/provider.go:34:2: ineffectual assignment to a
The reason will be displayed to describe this comment to others. Learn more.
⚠ shadow failed (.)
commands/steps/run.go:17:4: declaration of "step" shadows declaration at line 13
updater/main.go:61:12: declaration of "err" shadows declaration at line 21
util/exec.go:54:5: declaration of "err" shadows declaration at line 40
util/provider/native.go:140:16: declaration of "err" shadows declaration at line 122
util/provider/share/share_linux.go:72:3: declaration of "err" shadows declaration at line 61
util/provider/share/share_linux.go:124:3: declaration of "err" shadows declaration at line 113
The reason will be displayed to describe this comment to others. Learn more.
⚠ staticcheck failed (.)
commands/build.go:52:9: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
commands/configure.go:91:2: redundant return statement (S1023)
commands/configure.go:97:2: var prettyJson should be prettyJSON (ST1003)
commands/configure.go:99:2: redundant return statement (S1023)
commands/dns.go:12:2: var DnsCmd should be DNSCmd (ST1003)
commands/inspect.go:12:2: type anything is unused (U1000)
commands/inspect.go:32:2: default case should be first or last in switch statement (ST1015)
commands/log.go:21:2: var logStart is unused (U1000)
commands/log.go:22:2: var logEnd is unused (U1000)
commands/log.go:23:2: var logLimit is unused (U1000)
commands/server/server.go:28:7: const name is unused (U1000)
commands/server/svc_unix.go:7:2: redundant return statement (S1023)
generators/hooks/build/user.go:104:6: x509.IsEncryptedPEMBlock has been deprecated since Go 1.16 because it shouldn't be used: Legacy PEM encryption as specified in RFC 1423 is insecure by design. Since it does not authenticate the ciphertext, it is vulnerable to padding oracle attacks that can let an attacker recover the plaintext. (SA1019)
generators/hooks/build/user.go:120:16: x509.DecryptPEMBlock has been deprecated since Go 1.16 because it shouldn't be used: Legacy PEM encryption as specified in RFC 1423 is insecure by design. Since it does not authenticate the ciphertext, it is vulnerable to padding oracle attacks that can let an attacker recover the plaintext. (SA1019)
generators/hooks/code/configure.go:29:3: struct field DnsEntries should be DNSEntries (ST1003)
main.go:140:7: should omit comparison to bool constant, can be simplified to found (S1002)
main.go:146:7: should omit comparison to bool constant, can be simplified to found (S1002)
models/config.go:27:2: struct field SshKey should be SSHKey (ST1003)
models/config.go:28:2: struct field SshEncryptedKeys should be SSHEncryptedKeys (ST1003)
processors/code/build.go:144:9: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
processors/code/code.go:60:9: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
processors/component/clean.go:76:3: should use 'return !strings.Contains(err.Error(), "host is down")' instead of 'if strings.Contains(err.Error(), "host is down") { return false }; return true' (S1008)
processors/env/destroy.go:49:2: empty branch (SA9003)
processors/env/unmount.go:73:6: func mounted is unused (U1000)
processors/log/log.go:154:25: os.Kill cannot be trapped (did you mean syscall.SIGTERM?) (SA1016)
processors/platform/mist.go:41:25: os.Kill cannot be trapped (did you mean syscall.SIGTERM?) (SA1016)
processors/provider/bridge/bridge.go:87:5: should use 'return i.Flags&net.FlagUp == net.FlagUp' instead of 'if i.Flags&net.FlagUp != net.FlagUp { return false }; return true' (S1008)
processors/provider/stop.go:22:2: empty branch (SA9003)
updater/main.go:15:1: comment on exported var VERSION should be of the form "VERSION ..." (ST1022)
util/config/dirs.go:119:16: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
util/config/vars.go:23:9: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
util/display/command_error.go:75:19: boxfile.NewFromPath is deprecated: Use NewFromFile instead, which diferentiates between a missing or invalid boxfile (SA1019)
util/display/docker_percent.go:152:3: empty branch (SA9003)
util/display/docker_percent.go:152:6: HasPrefix is a pure function but its return value is ignored (SA4017)
util/display/logs.go:68:80: printf-style function with dynamic format string and no further arguments should use print-style function instead (SA1006)
util/display/logs.go:71:2: redundant return statement (S1023)
util/display/logs.go:110:80: printf-style function with dynamic format string and no further arguments should use print-style function instead (SA1006)
util/display/logs.go:114:2: redundant return statement (S1023)
util/display/messages.go:10:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:30:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:43:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:55:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:120:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:130:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:141:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:152:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:163:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:176:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:187:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:229:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:246:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:258:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:268:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:281:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:291:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:298:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:338:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:349:24: unnecessary use of fmt.Sprintf (S1039)
util/display/messages.go:359:24: unnecessary use of fmt.Sprintf (S1039)
util/display/prefix.go:47:5: should omit comparison to bool constant, can be simplified to p.firstLine (S1002)
util/display/prefix.go:62:13: should omit comparison to bool constant, can be simplified to p.escapeSeq (S1002)
util/display/prefix.go:69:20: should omit comparison to bool constant, can be simplified to !p.newLine (S1002)
util/display/prefix.go:71:27: should omit comparison to bool constant, can be simplified to !p.newLine (S1002)
util/display/prefix.go:85:7: should omit comparison to bool constant, can be simplified to p.newLine (S1002)
util/display/summary.go:19:18: should use raw string (`...`) with regexp.MustCompile to avoid having to escape twice (S1007)
util/provider/bridge/bridge.go:11:1: comment on exported type Response should be of the form "Response ..." (with optional leading article) (ST1021)
util/provider/dockermachine.go:39:39: error should be returned as the last argument (ST1008)
util/provider/dockermachine.go:496:30: methods on the same type should have the same receiver name (seen 1x "dockermachine", 39x "machine") (ST1016)
util/provider/dockermachine.go:1009:6: func dockerMachineURL is unused (U1000)
util/provider/dockermachine_mount.go:54:3: should use 'return !strings.Contains(string(output), "No such file or directory")' instead of 'if strings.Contains(string(output), "No such file or directory") { return false }; return true' (S1008)
util/provider/dockermachine_mount.go:311:6: func setupCoreUtilsScript is unused (U1000)
util/provider/native.go:28:31: error should be returned as the last argument (ST1008)
util/provider/provider.go:43:2: var verbose is unused (U1000)
util/provider/provider.go:57:15: error should be returned as the last argument (ST1008)
util/provider/share/share_unix.go:39:5: var sudoGid should be sudoGID (ST1003)
util/watch/watch.go:22:5: var ctimeAvailable is unused (U1000)
util/watch/watch.go:127:6: func ctimeCheck is unused (U1000)
util/watch/watch.go:144:6: func ctime is unused (U1000)
util/watch/watch.go:162:2: should replace loop with ignoreFile = append(ignoreFile, stringFields...) (S1011)
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ misspell failed (.)
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ gocyclo failed (.)
6 function(s) exceeding a complexity of 15
Show Detail
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ ineffassign failed (.)
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ errcheck failed (.)
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ gosec failed (.)
Show Detail
Code Reference
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ shadow failed (.)
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ staticcheck failed (.)
Checks Document
46c215bThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⚠ Failure Summary