Report for github.com/qaisjp/go-discord-irc
A+ Excellent! Found 13 issues across 23 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
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!
gocyclo73%
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.
-
-
go-discord-irc/bridge/discord.go
- Line 96: warning: cyclomatic complexity 22 of function (*discordBot).publishMessage() is high (> 15) (gocyclo)
- Line 254: warning: cyclomatic complexity 16 of function (*discordBot).ParseText() is high (> 15) (gocyclo)
-
-
go-discord-irc/bridge/irc_manager.go
- Line 141: warning: cyclomatic complexity 20 of function (*IRCManager).HandleUser() is high (> 15) (gocyclo)
-
-
go-discord-irc/bridge/irc_listener.go
- Line 123: warning: cyclomatic complexity 16 of function (*ircListener).OnJoinQuitCallback() is high (> 15) (gocyclo)
-
-
go-discord-irc/main.go
- Line 22: warning: cyclomatic complexity 20 of function main() is high (> 15) (gocyclo)
-
-
go-discord-irc/irc/format/markdown.go
- Line 5: warning: cyclomatic complexity 24 of function BlocksToMarkdown() is high (> 15) (gocyclo)
-
-
go-discord-irc/bridge/bridge.go
- Line 123: warning: cyclomatic complexity 25 of function (*Bridge).SetChannelMappings() is high (> 15) (gocyclo)
- Line 367: warning: cyclomatic complexity 20 of function (*Bridge).loop() is high (> 15) (gocyclo)
golint60%
Golint is a linter for Go source code.
-
-
go-discord-irc/dstate/dstate.go
- Line 6: warning: exported function ChannelMessage should have comment or be unexported (golint)
-
-
go-discord-irc/irc/format/block.go
- Line 7: warning: exported type Block should have comment or be unexported (golint)
- Line 13: warning: exported var Empty should have comment or be unexported (golint)
- Line 15: warning: exported function NewBlock should have comment or be unexported (golint)
- Line 27: warning: exported function NewColorBlock should have comment or be unexported (golint)
- Line 34: warning: exported method Block.Equals should have comment or be unexported (golint)
- Line 34: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 43: warning: exported method Block.IsPlain should have comment or be unexported (golint)
- Line 43: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 52: warning: exported method Block.HasSameColor should have comment or be unexported (golint)
- Line 52: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 59: warning: exported method Block.GetColorString should have comment or be unexported (golint)
- Line 59: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 73: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 86: warning: exported method Block.SetField should have comment or be unexported (golint)
- Line 86: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
- Line 94: warning: exported method Block.GetField should have comment or be unexported (golint)
- Line 94: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
-
-
go-discord-irc/irc/format/format.go
- Line 39: warning: exported var Keys should have comment or be unexported (golint)
- Line 45: warning: exported function StripCodes should have comment or be unexported (golint)
- Line 49: warning: exported function StripColor should have comment or be unexported (golint)
- Line 87: warning: exported function Parse should have comment or be unexported (golint)
-
-
go-discord-irc/irc/varys/client_net.go
- Line 12: warning: exported function NewNetClient should have comment or be unexported (golint)
-
-
go-discord-irc/main.go
- Line 303: warning: exported function SetLogDebug should have comment or be unexported (golint)
-
-
go-discord-irc/irc/nick/funcs.go
- Line 17: warning: exported function IsHostChar should have comment or be unexported (golint)
- Line 18: warning: exported function IsUserChar should have comment or be unexported (golint)
- Line 19: warning: exported function IsChanPrefix should have comment or be unexported (golint)
- Line 20: warning: exported function IsChanChar should have comment or be unexported (golint)
- Line 21: warning: exported function IsFakeChanChar should have comment or be unexported (golint)
- Line 22: warning: exported function IsKWildChar should have comment or be unexported (golint)
- Line 23: warning: exported function IsMWildChar should have comment or be unexported (golint)
- Line 24: warning: exported function IsNickChar should have comment or be unexported (golint)
- Line 25: warning: exported function IsFakeNickChar should have comment or be unexported (golint)
- Line 26: warning: exported function IsServChar should have comment or be unexported (golint)
- Line 27: warning: exported function IsIdChar should have comment or be unexported (golint)
- Line 28: warning: exported function IsLetter should have comment or be unexported (golint)
- Line 29: warning: exported function IsCntrl should have comment or be unexported (golint)
- Line 30: warning: exported function IsAlpha should have comment or be unexported (golint)
- Line 31: warning: exported function IsSpace should have comment or be unexported (golint)
- Line 32: warning: exported function IsLower should have comment or be unexported (golint)
- Line 33: warning: exported function IsUpper should have comment or be unexported (golint)
- Line 34: warning: exported function IsDigit should have comment or be unexported (golint)
- Line 35: warning: exported function IsXDigit should have comment or be unexported (golint)
- Line 38: warning: exported function IsAlNum should have comment or be unexported (golint)
- Line 39: warning: exported function IsPrint should have comment or be unexported (golint)
- Line 40: warning: exported function IsAscii should have comment or be unexported (golint)
- Line 41: warning: exported function IsGraph should have comment or be unexported (golint)
- Line 42: warning: exported function IsPunct should have comment or be unexported (golint)
- Line 44: warning: exported function IsNonEOS should have comment or be unexported (golint)
- Line 45: warning: exported function IsEol should have comment or be unexported (golint)
-
-
go-discord-irc/irc/format/markdown.go
- Line 5: warning: exported function BlocksToMarkdown should have comment or be unexported (golint)
-
-
go-discord-irc/irc/varys/server.go
- Line 3: warning: exported function NewServer should have comment or be unexported (golint)
-
-
go-discord-irc/irc/varys/varys.go
- Line 15: warning: exported type Varys should have comment or be unexported (golint)
- Line 20: warning: exported function NewVarys should have comment or be unexported (golint)
- Line 37: warning: exported type Client should have comment or be unexported (golint)
- Line 52: warning: exported type SetupParams should have comment or be unexported (golint)
- Line 61: warning: exported method Varys.Setup should have comment or be unexported (golint)
- Line 66: warning: exported method Varys.GetUIDToNicks should have comment or be unexported (golint)
- Line 76: warning: exported type ConnectParams should have comment or be unexported (golint)
- Line 89: warning: exported method Varys.Connect should have comment or be unexported (golint)
- Line 129: warning: exported type QuitParams should have comment or be unexported (golint)
- Line 134: warning: exported method Varys.QuitIfConnected should have comment or be unexported (golint)
- Line 145: warning: exported type InterpolationParams should have comment or be unexported (golint)
- Line 148: warning: exported type SendRawParams should have comment or be unexported (golint)
- Line 155: warning: exported method Varys.SendRaw should have comment or be unexported (golint)
- Line 167: warning: exported method Varys.GetNick should have comment or be unexported (golint)
- Line 174: warning: exported method Varys.Connected should have comment or be unexported (golint)
- Line 182: warning: exported type NickParams should have comment or be unexported (golint)
- Line 187: warning: exported method Varys.Nick should have comment or be unexported (golint)
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!
misspell95%
Misspell Finds commonly misspelled English words
-
-
go-discord-irc/bridge/discord.go
- Line 128: warning: "thereotically" is a misspelling of "theoretically" (misspell)
- Line 223: warning: "thereotically" is a misspelling of "theoretically" (misspell)