Showing 112 of 112 total issues
exported const ReviewTestcase should have comment (or a comment on this block) or be unexported Open
Open
ReviewTestcase = 1 + iota
- Exclude checks
struct field ArtifactsUrl should be ArtifactsURL Open
Open
ArtifactsUrl string
- Exclude checks
struct field AssigneeId should be AssigneeID Open
Open
AssigneeId int // TODO add foreign key
- Exclude checks
struct field Id should be ID Open
Open
Id string `json:"id"`
- Exclude checks
if block ends with a return statement, so drop this else and outdent its block Open
Open
} else {
- Exclude checks
exported method Build.Validate should have comment or be unexported Open
Open
func (build *Build) Validate() map[string]string {
- Exclude checks
comment on exported type JSTreeTestSuiteWrapper should be of the form "JSTreeTestSuiteWrapper ..." (with optional leading article) Open
Open
// Wrapper type for print out JSON format used in JSTree
- Exclude checks
exported type JSTreeTestSuiteWrapperWithChildren should have comment or be unexported Open
Open
type JSTreeTestSuiteWrapperWithChildren struct {
- Exclude checks
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) Open
Open
} else {
- Exclude checks
struct field IdByTool should be IDByTool Open
Open
IdByTool string // for example, jenkins builds id is string
- Exclude checks
exported type Issue should have comment or be unexported Open
Open
type Issue struct {
- Exclude checks
struct field Id should be ID Open
Open
Id string `json:"id"`
- Exclude checks