Preparing report...

Report for github.com/moov-io/paygate

A+    Excellent!    Found 94 issues across 211 files

Tweet

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!


gocyclo96%

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.

    • paygate/pkg/client/api_transfers.go
    • Line 349: warning: cyclomatic complexity 27 of function (*TransfersApiService).GetTransfers() is high (> 15) (gocyclo)
    • Line 46: warning: cyclomatic complexity 17 of function (*TransfersApiService).AddTransfer() is high (> 15) (gocyclo)

golint57%

Golint is a linter for Go source code.

    • paygate/pkg/config/validation.go
    • Line 11: warning: exported type Validation should have comment or be unexported (golint)
    • Line 15: warning: exported method Validation.Validate should have comment or be unexported (golint)
    • Line 22: warning: exported type MicroDeposits should have comment or be unexported (golint)
    • Line 33: warning: exported method MicroDeposits.Validate should have comment or be unexported (golint)
    • Line 43: warning: exported type Source should have comment or be unexported (golint)
    • Line 49: warning: exported method Source.Validate should have comment or be unexported (golint)
    • paygate/x/route/route.go
    • Line 27: warning: exported var IdempotentRecorder should have comment or be unexported (golint)
    • Line 29: warning: comment on exported var Histogram should be of the form "Histogram ..." (golint)
    • Line 36: warning: exported type Responder should have comment or be unexported (golint)
    • Line 48: warning: exported function NewResponder should have comment or be unexported (golint)
    • Line 69: warning: exported method Responder.Respond should have comment or be unexported (golint)
    • Line 79: warning: exported method Responder.Problem should have comment or be unexported (golint)
    • paygate/pkg/validation/microdeposits/router.go
    • Line 29: warning: exported type Router should have comment or be unexported (golint)
    • Line 35: warning: exported function NewRouter should have comment or be unexported (golint)
    • Line 64: warning: exported method Router.RegisterRoutes should have comment or be unexported (golint)
    • Line 70: warning: exported function InitiateMicroDeposits should have comment or be unexported (golint)
    • Line 148: warning: exported function GetMicroDeposits should have comment or be unexported (golint)
    • Line 171: warning: exported function GetAccountMicroDeposits should have comment or be unexported (golint)
    • Line 194: warning: exported function NotImplemented should have comment or be unexported (golint)
    • paygate/x/mask/password.go
    • Line 13: warning: exported function Password should have comment or be unexported (golint)
    • Line 16: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • paygate/examples/common/common.go
    • Line 37: warning: exported function CreateCustomer should have comment or be unexported (golint)
    • Line 67: warning: exported function ApproveCustomer should have comment or be unexported (golint)
    • Line 92: warning: exported function CreateAccount should have comment or be unexported (golint)
    • Line 122: warning: exported function ApproveAccount should have comment or be unexported (golint)
    • Line 146: warning: exported function InitiateMicroDeposits should have comment or be unexported (golint)
    • Line 182: warning: exported function GetMicroDeposits should have comment or be unexported (golint)
    • Line 201: warning: exported function MakeTransfer should have comment or be unexported (golint)
    • Line 244: warning: exported function GetTransfer should have comment or be unexported (golint)
    • Line 263: warning: exported function GetCustomerAccounts should have comment or be unexported (golint)
    • Line 282: warning: exported function VerifyMicroDeposits should have comment or be unexported (golint)
    • Line 309: warning: exported function TriggerCutOff should have comment or be unexported (golint)
    • Line 323: warning: exported function PrintServerFiles should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/transform/gpg.go
    • Line 23: warning: exported type GPGEncryption should have comment or be unexported (golint)
    • Line 28: warning: exported function NewGPGEncryptor should have comment or be unexported (golint)
    • Line 77: warning: exported method GPGEncryption.Transform should have comment or be unexported (golint)
    • paygate/pkg/transfers/inbound/mock_scheduler.go
    • Line 7: warning: exported type MockScheduler should have comment or be unexported (golint)
    • Line 11: warning: exported method MockScheduler.Start should have comment or be unexported (golint)
    • Line 15: warning: exported method MockScheduler.Shutdown should have comment or be unexported (golint)
    • paygate/pkg/upload/ftp.go
    • Line 77: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 132: warning: exported method FTPTransferAgent.Ping should have comment or be unexported (golint)
    • Line 162: warning: exported method FTPTransferAgent.Close should have comment or be unexported (golint)
    • Line 177: warning: exported method FTPTransferAgent.InboundPath should have comment or be unexported (golint)
    • Line 181: warning: exported method FTPTransferAgent.OutboundPath should have comment or be unexported (golint)
    • Line 185: warning: exported method FTPTransferAgent.ReturnPath should have comment or be unexported (golint)
    • Line 189: warning: exported method FTPTransferAgent.Hostname should have comment or be unexported (golint)
    • Line 196: warning: exported method FTPTransferAgent.Delete should have comment or be unexported (golint)
    • Line 211: warning: comment on exported method FTPTransferAgent.UploadFile should be of the form "UploadFile ..." (golint)
    • Line 245: warning: exported method FTPTransferAgent.GetInboundFiles should have comment or be unexported (golint)
    • Line 249: warning: exported method FTPTransferAgent.GetReturnFiles should have comment or be unexported (golint)
    • paygate/pkg/transfers/fundflow/first_party.go
    • Line 24: warning: comment on exported type FirstParty should be of the form "FirstParty ..." (with optional leading article) (golint)
    • Line 41: warning: exported function NewFirstPerson should have comment or be unexported (golint)
    • Line 49: warning: exported method FirstParty.Originate should have comment or be unexported (golint)
    • Line 98: warning: exported method FirstParty.HandleReturn should have comment or be unexported (golint)
    • paygate/pkg/config/config.go
    • Line 20: warning: exported type Config should have comment or be unexported (golint)
    • Line 38: warning: exported type Logging should have comment or be unexported (golint)
    • Line 43: warning: exported function Empty should have comment or be unexported (golint)
    • Line 61: warning: exported function FromFile should have comment or be unexported (golint)
    • Line 77: warning: exported function Read should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/notify/email.go
    • Line 25: warning: exported type Email should have comment or be unexported (golint)
    • Line 30: warning: exported type EmailTemplateData should have comment or be unexported (golint)
    • Line 48: warning: exported function NewEmail should have comment or be unexported (golint)
    • Line 93: warning: exported method Email.Info should have comment or be unexported (golint)
    • Line 101: warning: exported method Email.Critical should have comment or be unexported (golint)
    • paygate/pkg/upload/sftp.go
    • Line 37: warning: exported type SFTPTransferAgent should have comment or be unexported (golint)
    • Line 70: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 192: warning: exported method SFTPTransferAgent.Ping should have comment or be unexported (golint)
    • Line 225: warning: exported method SFTPTransferAgent.Close should have comment or be unexported (golint)
    • Line 238: warning: exported method SFTPTransferAgent.InboundPath should have comment or be unexported (golint)
    • Line 242: warning: exported method SFTPTransferAgent.OutboundPath should have comment or be unexported (golint)
    • Line 246: warning: exported method SFTPTransferAgent.ReturnPath should have comment or be unexported (golint)
    • Line 250: warning: exported method SFTPTransferAgent.Hostname should have comment or be unexported (golint)
    • Line 257: warning: exported method SFTPTransferAgent.Delete should have comment or be unexported (golint)
    • Line 278: warning: comment on exported method SFTPTransferAgent.UploadFile should be of the form "UploadFile ..." (golint)
    • Line 319: warning: exported method SFTPTransferAgent.GetInboundFiles should have comment or be unexported (golint)
    • Line 323: warning: exported method SFTPTransferAgent.GetReturnFiles should have comment or be unexported (golint)
    • Line 367: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • paygate/pkg/transfers/pipeline/notify/multi.go
    • Line 22: warning: exported function NewMultiSender should have comment or be unexported (golint)
    • Line 60: warning: exported method MultiSender.Info should have comment or be unexported (golint)
    • Line 74: warning: exported method MultiSender.Critical should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/notify/pagerduty.go
    • Line 16: warning: exported type PagerDuty should have comment or be unexported (golint)
    • Line 22: warning: exported function NewPagerDuty should have comment or be unexported (golint)
    • Line 34: warning: exported method PagerDuty.Ping should have comment or be unexported (golint)
    • Line 51: warning: exported method PagerDuty.Info should have comment or be unexported (golint)
    • Line 67: warning: exported method PagerDuty.Critical should have comment or be unexported (golint)
    • paygate/pkg/config/database.go
    • Line 13: warning: exported type Database should have comment or be unexported (golint)
    • Line 18: warning: exported type SQLite should have comment or be unexported (golint)
    • Line 22: warning: exported type MySQL should have comment or be unexported (golint)
    • Line 29: warning: exported method MySQL.GetPassword should have comment or be unexported (golint)
    • paygate/pkg/customers/mock_client.go
    • Line 11: warning: exported type MockClient should have comment or be unexported (golint)
    • Line 20: warning: exported method MockClient.Ping should have comment or be unexported (golint)
    • Line 24: warning: exported method MockClient.Lookup should have comment or be unexported (golint)
    • Line 36: warning: exported method MockClient.FindAccount should have comment or be unexported (golint)
    • Line 46: warning: exported method MockClient.DecryptAccount should have comment or be unexported (golint)
    • Line 53: warning: exported method MockClient.LatestOFACSearch should have comment or be unexported (golint)
    • Line 60: warning: exported method MockClient.RefreshOFACSearch should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/publisher_mock.go
    • Line 11: warning: exported type MockPublisher should have comment or be unexported (golint)
    • Line 18: warning: exported function NewMockPublisher should have comment or be unexported (golint)
    • Line 25: warning: exported method MockPublisher.Upload should have comment or be unexported (golint)
    • Line 30: warning: exported method MockPublisher.Cancel should have comment or be unexported (golint)
    • Line 35: warning: exported method MockPublisher.Shutdown should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/notify/mock_sender.go
    • Line 3: warning: exported type MockSender should have comment or be unexported (golint)
    • Line 10: warning: exported method MockSender.Info should have comment or be unexported (golint)
    • Line 16: warning: exported method MockSender.Critical should have comment or be unexported (golint)
    • Line 22: warning: exported method MockSender.InfoWasCalled should have comment or be unexported (golint)
    • Line 26: warning: exported method MockSender.CriticalWasCalled should have comment or be unexported (golint)
    • Line 30: warning: exported method MockSender.CapturedMessage should have comment or be unexported (golint)
    • paygate/pkg/upload/file.go
    • Line 11: warning: exported type File should have comment or be unexported (golint)
    • Line 16: warning: exported method File.Close should have comment or be unexported (golint)
    • paygate/pkg/upload/filename_template.go
    • Line 17: warning: exported type FilenameData should have comment or be unexported (golint)
    • Line 33: warning: exported function RenderACHFilename should have comment or be unexported (golint)
    • Line 55: warning: comment on exported function ACHFilenameSeq should be of the form "ACHFilenameSeq ..." (golint)
    • paygate/pkg/upload/mock_agent.go
    • Line 13: warning: exported type MockAgent should have comment or be unexported (golint)
    • Line 23: warning: exported method MockAgent.GetInboundFiles should have comment or be unexported (golint)
    • Line 30: warning: exported method MockAgent.GetReturnFiles should have comment or be unexported (golint)
    • Line 37: warning: exported method MockAgent.UploadFile should have comment or be unexported (golint)
    • Line 48: warning: exported method MockAgent.Delete should have comment or be unexported (golint)
    • Line 56: warning: exported method MockAgent.InboundPath should have comment or be unexported (golint)
    • Line 60: warning: exported method MockAgent.OutboundPath should have comment or be unexported (golint)
    • Line 64: warning: exported method MockAgent.ReturnPath should have comment or be unexported (golint)
    • Line 68: warning: exported method MockAgent.Hostname should have comment or be unexported (golint)
    • Line 72: warning: exported method MockAgent.Ping should have comment or be unexported (golint)
    • Line 76: warning: exported method MockAgent.Close should have comment or be unexported (golint)
    • paygate/pkg/achx/files.go
    • Line 20: warning: exported type Source should have comment or be unexported (golint)
    • Line 28: warning: exported type Destination should have comment or be unexported (golint)
    • Line 36: warning: exported type Options should have comment or be unexported (golint)
    • Line 50: warning: exported function ConstructFile should have comment or be unexported (golint)
    • paygate/pkg/transfers/fundflow/mock_strategy.go
    • Line 12: warning: exported type MockStrategy should have comment or be unexported (golint)
    • Line 17: warning: exported method MockStrategy.Originate should have comment or be unexported (golint)
    • Line 24: warning: exported method MockStrategy.HandleReturn should have comment or be unexported (golint)
    • paygate/pkg/transfers/inbound/scheduler.go
    • Line 18: warning: exported type Scheduler should have comment or be unexported (golint)
    • Line 23: warning: exported type PeriodicScheduler should have comment or be unexported (golint)
    • Line 36: warning: exported function NewPeriodicScheduler should have comment or be unexported (golint)
    • Line 44: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 64: warning: exported method PeriodicScheduler.Shutdown should have comment or be unexported (golint)
    • Line 71: warning: exported method PeriodicScheduler.Start should have comment or be unexported (golint)
    • paygate/x/route/paths.go
    • Line 1: warning: package comment should be of the form "Package route ..." (golint)
    • Line 12: warning: exported function ReadPathID should have comment or be unexported (golint)
    • Line 21: warning: exported function GetHeaderValue should have comment or be unexported (golint)
    • Line 24: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • paygate/pkg/config/odfi.go
    • Line 62: warning: exported method ODFI.FilenameTemplate should have comment or be unexported (golint)
    • Line 69: warning: exported method ODFI.SplitAllowedIPs should have comment or be unexported (golint)
    • Line 76: warning: exported method ODFI.Validate should have comment or be unexported (golint)
    • Line 92: warning: exported type Gateway should have comment or be unexported (golint)
    • Line 99: warning: exported type Cutoffs should have comment or be unexported (golint)
    • Line 104: warning: exported method Cutoffs.Location should have comment or be unexported (golint)
    • Line 112: warning: exported method Cutoffs.Validate should have comment or be unexported (golint)
    • Line 122: warning: exported type FTP should have comment or be unexported (golint)
    • Line 132: warning: exported method FTP.CAFile should have comment or be unexported (golint)
    • Line 139: warning: exported method FTP.Timeout should have comment or be unexported (golint)
    • Line 146: warning: exported method FTP.DisableEPSV should have comment or be unexported (golint)
    • Line 161: warning: exported type SFTP should have comment or be unexported (golint)
    • Line 174: warning: exported method SFTP.Timeout should have comment or be unexported (golint)
    • Line 181: warning: exported method SFTP.MaxConnections should have comment or be unexported (golint)
    • Line 188: warning: exported method SFTP.PacketSize should have comment or be unexported (golint)
    • Line 205: warning: exported type Inbound should have comment or be unexported (golint)
    • Line 209: warning: exported type FileConfig should have comment or be unexported (golint)
    • Line 217: warning: exported method FileConfig.Validate should have comment or be unexported (golint)
    • Line 224: warning: exported type BatchHeader should have comment or be unexported (golint)
    • Line 228: warning: exported method BatchHeader.Validate should have comment or be unexported (golint)
    • Line 235: warning: exported type Addendum should have comment or be unexported (golint)
    • Line 239: warning: exported type Storage should have comment or be unexported (golint)
    • Line 255: warning: exported type Local should have comment or be unexported (golint)
    • paygate/pkg/database/sqlite.go
    • Line 124: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 181: warning: exported method TestSQLiteDB.Close should have comment or be unexported (golint)
    • paygate/pkg/transfers/mock_repository.go
    • Line 13: warning: exported type MockRepository should have comment or be unexported (golint)
    • Line 25: warning: exported method MockRepository.GetTransfer should have comment or be unexported (golint)
    • Line 35: warning: exported method MockRepository.UpdateTransferStatus should have comment or be unexported (golint)
    • Line 39: warning: exported method MockRepository.WriteUserTransfer should have comment or be unexported (golint)
    • Line 47: warning: exported method MockRepository.SaveReturnCode should have comment or be unexported (golint)
    • Line 55: warning: exported method MockRepository.LookupTransferFromReturn should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/notify/slack.go
    • Line 19: warning: exported type Slack should have comment or be unexported (golint)
    • Line 24: warning: exported function NewSlack should have comment or be unexported (golint)
    • Line 43: warning: exported method Slack.Info should have comment or be unexported (golint)
    • Line 48: warning: exported method Slack.Critical should have comment or be unexported (golint)
    • paygate/internal/gpgx/keys.go
    • Line 50: warning: exported function Encrypt should have comment or be unexported (golint)
    • Line 95: warning: exported function Sign should have comment or be unexported (golint)
    • Line 108: warning: exported function Decrypt should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/notify/notify.go
    • Line 11: warning: exported type Direction should have comment or be unexported (golint)
    • Line 14: warning: exported const Upload should have comment (or a comment on this block) or be unexported (golint)
    • Line 18: warning: exported type Message should have comment or be unexported (golint)
    • Line 25: warning: exported type Sender should have comment or be unexported (golint)
    • paygate/pkg/database/mysql.go
    • Line 145: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 194: warning: exported method TestMySQLDB.Close should have comment or be unexported (golint)
    • paygate/pkg/transfers/inbound/mock_processor.go
    • Line 11: warning: exported type MockProcessor should have comment or be unexported (golint)
    • Line 15: warning: exported method MockProcessor.Type should have comment or be unexported (golint)
    • Line 19: warning: exported method MockProcessor.Handle should have comment or be unexported (golint)
    • paygate/pkg/stream/stream.go
    • Line 22: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 25: warning: exported function Topic should have comment or be unexported (golint)
    • Line 29: warning: exported function Subscription should have comment or be unexported (golint)
    • paygate/pkg/config/customers.go
    • Line 11: warning: exported type Customers should have comment or be unexported (golint)
    • Line 17: warning: exported method Customers.Validate should have comment or be unexported (golint)
    • Line 24: warning: exported type Accounts should have comment or be unexported (golint)
    • Line 28: warning: exported type Decryptor should have comment or be unexported (golint)
    • Line 32: warning: exported method Decryptor.Validate should have comment or be unexported (golint)
    • Line 39: warning: exported type Symmetric should have comment or be unexported (golint)
    • paygate/pkg/config/transfers.go
    • Line 13: warning: exported type Transfers should have comment or be unexported (golint)
    • Line 17: warning: exported method Transfers.Validate should have comment or be unexported (golint)
    • Line 24: warning: exported type Limits should have comment or be unexported (golint)
    • Line 28: warning: exported method Limits.Validate should have comment or be unexported (golint)
    • Line 35: warning: exported type FixedLimits should have comment or be unexported (golint)
    • Line 45: warning: exported method FixedLimits.Validate should have comment or be unexported (golint)
    • Line 55: warning: exported method FixedLimits.OverSoftLimit should have comment or be unexported (golint)
    • Line 59: warning: exported method FixedLimits.OverHardLimit should have comment or be unexported (golint)
    • paygate/pkg/transfers/limiter/limiter.go
    • Line 15: warning: exported var ErrReviewableTransfer should have comment or be unexported (golint)
    • Line 19: warning: exported type Checker should have comment or be unexported (golint)
    • Line 23: warning: exported function New should have comment or be unexported (golint)
    • paygate/pkg/transfers/router.go
    • Line 35: warning: exported type Router should have comment or be unexported (golint)
    • Line 49: warning: exported function NewRouter should have comment or be unexported (golint)
    • Line 76: warning: exported method Router.RegisterRoutes should have comment or be unexported (golint)
    • Line 135: warning: exported function GetTransfers should have comment or be unexported (golint)
    • Line 161: warning: exported function CreateTransfer should have comment or be unexported (golint)
    • Line 266: warning: exported function SaveTraceNumbers should have comment or be unexported (golint)
    • Line 306: warning: exported function GetFundflowSource should have comment or be unexported (golint)
    • Line 326: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 331: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 338: warning: exported function GetFundflowDestination should have comment or be unexported (golint)
    • Line 358: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 363: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 370: warning: exported function GetUserTransfer should have comment or be unexported (golint)
    • Line 387: warning: exported function DeleteUserTransfer should have comment or be unexported (golint)
    • paygate/pkg/config/pipeline.go
    • Line 17: warning: exported var DefaultEmailTemplate should have comment or be unexported (golint)
    • Line 28: warning: exported type Pipeline should have comment or be unexported (golint)
    • Line 37: warning: exported method Pipeline.Validate should have comment or be unexported (golint)
    • Line 59: warning: exported type PreUpload should have comment or be unexported (golint)
    • Line 63: warning: exported method PreUpload.Validate should have comment or be unexported (golint)
    • Line 73: warning: exported type GPG should have comment or be unexported (golint)
    • Line 78: warning: exported type Signer should have comment or be unexported (golint)
    • Line 83: warning: exported method Signer.Password should have comment or be unexported (golint)
    • Line 87: warning: exported type Output should have comment or be unexported (golint)
    • Line 91: warning: exported method Output.Validate should have comment or be unexported (golint)
    • Line 95: warning: exported type Merging should have comment or be unexported (golint)
    • Line 100: warning: exported method Merging.Validate should have comment or be unexported (golint)
    • Line 104: warning: exported type FlattenBatches should have comment or be unexported (golint)
    • Line 106: warning: exported type AuditTrail should have comment or be unexported (golint)
    • Line 111: warning: exported method AuditTrail.Validate should have comment or be unexported (golint)
    • Line 121: warning: exported type StreamPipeline should have comment or be unexported (golint)
    • Line 126: warning: exported method StreamPipeline.Validate should have comment or be unexported (golint)
    • Line 141: warning: exported type InMemPipeline should have comment or be unexported (golint)
    • Line 145: warning: exported type KafkaPipeline should have comment or be unexported (golint)
    • Line 151: warning: exported type PipelineNotifications should have comment or be unexported (golint)
    • Line 157: warning: exported method PipelineNotifications.Validate should have comment or be unexported (golint)
    • Line 175: warning: exported type Email should have comment or be unexported (golint)
    • Line 191: warning: exported method Email.Tmpl should have comment or be unexported (golint)
    • Line 198: warning: exported type PagerDuty should have comment or be unexported (golint)
    • Line 204: warning: exported type Slack should have comment or be unexported (golint)
    • Line 208: warning: exported method Slack.Validate should have comment or be unexported (golint)
    • paygate/pkg/transfers/inbound/processor.go
    • Line 16: warning: exported type FileProcessor should have comment or be unexported (golint)
    • Line 23: warning: exported type Processors should have comment or be unexported (golint)
    • Line 25: warning: exported function SetupProcessors should have comment or be unexported (golint)
    • Line 29: warning: exported method Processors.HandleAll should have comment or be unexported (golint)
    • Line 42: warning: exported function ProcessFiles should have comment or be unexported (golint)
    • paygate/x/schedule/cutoff.go
    • Line 25: warning: exported function ForCutoffTimes should have comment or be unexported (golint)
    • Line 37: warning: exported method CutoffTimes.Stop should have comment or be unexported (golint)
    • paygate/x/trace/http.go
    • Line 14: warning: exported function DecorateHttpRequest should have comment or be unexported (golint)
    • Line 33: warning: exported function FromRequest should have comment or be unexported (golint)
    • paygate/pkg/organization/mock_repository.go
    • Line 9: warning: exported type MockRepository should have comment or be unexported (golint)
    • Line 14: warning: exported method MockRepository.GetConfig should have comment or be unexported (golint)
    • Line 21: warning: exported method MockRepository.UpdateConfig should have comment or be unexported (golint)
    • paygate/pkg/organization/router.go
    • Line 19: warning: exported type Router should have comment or be unexported (golint)
    • Line 24: warning: exported function NewRouter should have comment or be unexported (golint)
    • Line 31: warning: exported method Router.RegisterRoutes should have comment or be unexported (golint)
    • paygate/pkg/transfers/pipeline/aggregate.go
    • Line 57: warning: exported function NewAggregator should have comment or be unexported (golint)
    • Line 135: warning: exported method XferAggregator.Start should have comment or be unexported (golint)
    • Line 163: warning: exported method XferAggregator.Shutdown should have comment or be unexported (golint)
    • Line 303: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 317: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • paygate/pkg/transfers/pipeline/merging.go
    • Line 42: warning: exported function NewMerging should have comment or be unexported (golint)
    • Line 110: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • paygate/pkg/validation/microdeposits/micro_deposits.go
    • Line 49: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 54: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • Line 69: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
    • paygate/pkg/transfers/pipeline/mock_xfer_merging.go
    • Line 11: warning: exported type MockXferMerging should have comment or be unexported (golint)
    • Line 19: warning: exported method MockXferMerging.HandleXfer should have comment or be unexported (golint)
    • Line 24: warning: exported method MockXferMerging.HandleCancel should have comment or be unexported (golint)
    • Line 29: warning: exported method MockXferMerging.WithEachMerged should have comment or be unexported (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!