Omit explicit bool check

context/context.go

@@ -333,7 +333,7 @@ func (ctx *Context) findPackageChild(ck *Package) []*Package {
 		if pkg == ck {
 			continue
 		}
-		if pkg.inVendor == false {
+		if !pkg.inVendor {
 			continue
 		}
 		if pkg.Status.Presence == PresenceTree {
@@ -351,10 +351,10 @@ func (ctx *Context) findPackageChild(ck *Package) []*Package {
 func (ctx *Context) findPackageParentTree(ck *Package) []string {
 	out := make([]string, 0, 1)
 	for _, pkg := range ctx.Package {
-		if pkg.inVendor == false {
+		if !pkg.inVendor {
 			continue
 		}
-		if pkg.IncludeTree == false || pkg == ck {
+		if !pkg.IncludeTree || pkg == ck {
 			continue
 		}
 		// pkg.Path = github.com/usera/pkg, tree = true

context/context_test.go

@@ -1065,7 +1065,7 @@ func TestRemove(t *testing.T) {
 	if err != nil {
 		t.Fatal("vendor folder should still be present", err)
 	}
-	if vi.IsDir() == false {
+	if !vi.IsDir() {
 		t.Fatal("vendor folder is not a dir")
 	}
 }

context/label.go

@@ -74,7 +74,7 @@ func (item *labelAnalysis) fillSections(buf *bytes.Buffer) {
 			if err != nil {
 				value = -1
 			}
-			if isBreak(r) == false {
+			if !isBreak(r) {
 				r = 0
 			}
 			group.sections = append(group.sections, labelSection{
@@ -209,7 +209,7 @@ func FindLabel(version string, labels []Label) Label {
 			}
 			continue
 		}
-		if strings.HasPrefix(label.Text, version) == false {
+		if !strings.HasPrefix(label.Text, version) {
 			continue
 		}
 		remain := strings.TrimPrefix(label.Text, version)

context/modify.go

@@ -470,7 +470,7 @@ func (ctx *Context) modifyRemove(pkg *Package) error {
 		return nil
 	}
 	// Protect non-project paths from being removed.
-	if pathos.FileHasPrefix(pkg.Dir, ctx.RootDir) == false {
+	if !pathos.FileHasPrefix(pkg.Dir, ctx.RootDir) {
 		return nil
 	}
 	if pkg.Status.Location == LocationLocal {
@@ -569,7 +569,7 @@ func (ctx *Context) Check() []*Conflict {
 // Operation listed in the OpIndex field.
 func (ctx *Context) ResloveApply(cc []*Conflict) {
 	for _, c := range cc {
-		if c.Resolved == false {
+		if !c.Resolved {
 			continue
 		}
 		for i, op := range c.Operation {

context/path.go

@@ -24,7 +24,7 @@ func (ctx *Context) isStdLib(importPath string) (yes bool, err error) {
 	if fi == nil {
 		return
 	}
-	if fi.IsDir() == false {
+	if !fi.IsDir() {
 		return
 	}
 
@@ -54,7 +54,7 @@ func (ctx *Context) findImportDir(relative, importPath string) (dir, gopath stri
 			if err != nil {
 				continue
 			}
-			if fi.IsDir() == false {
+			if !fi.IsDir() {
 				continue
 			}
 			for _, gopath = range ctx.GopathList {
@@ -80,7 +80,7 @@ func (ctx *Context) findImportDir(relative, importPath string) (dir, gopath stri
 		if fi == nil {
 			continue
 		}
-		if fi.IsDir() == false {
+		if !fi.IsDir() {
 			continue
 		}
 
@@ -117,7 +117,7 @@ func findRoot(folder, vendorPath string) (root string, err error) {
 	for i := 0; i <= looplimit; i++ {
 		test := filepath.Join(folder, vendorPath)
 		_, err := os.Stat(test)
-		if os.IsNotExist(err) == false {
+		if !os.IsNotExist(err) {
 			return folder, nil
 		}
 		nextFolder := filepath.Clean(filepath.Join(folder, ".."))
@@ -146,7 +146,7 @@ func hasGoFileInFolder(folder string) (bool, error) {
 		return false, err
 	}
 	for _, fi := range fl {
-		if fi.IsDir() == false && filepath.Ext(fi.Name()) == ".go" {
+		if !fi.IsDir() && filepath.Ext(fi.Name()) == ".go" {
 			return true, nil
 		}
 	}
@@ -210,7 +210,7 @@ func RemovePackage(path, root string, tree bool) error {
 		if len(fl) > 0 {
 			allAreLicense := true
 			for _, fi := range fl {
-				if isLicenseFile(fi.Name()) == false {
+				if !isLicenseFile(fi.Name()) {
 					allAreLicense = false
 					break
 				}

context/resolve.go

@@ -80,7 +80,7 @@ func (ctx *Context) loadPackage() error {
 func (ctx *Context) getFileTags(pathname string, f *ast.File) (tags *TagSet, imports []string, err error) {
 	_, filenameExt := filepath.Split(pathname)
 
-	if strings.HasSuffix(pathname, ".go") == false {
+	if !strings.HasSuffix(pathname, ".go") {
 		return nil, nil, nil
 	}
 	if f == nil {
@@ -151,12 +151,12 @@ func (ctx *Context) addFileImports(pathname, gopath string) (*Package, error) {
 	importPath = pathos.SlashToImportPath(importPath)
 	importPath = strings.Trim(importPath, "/")
 
-	if strings.HasSuffix(pathname, ".go") == false {
+	if !strings.HasSuffix(pathname, ".go") {
 		return nil, nil
 	}
 	// No need to add the same file more than once.
 	for _, pkg := range ctx.Package {
-		if pathos.FileStringEquals(pkg.Dir, dir) == false {
+		if !pathos.FileStringEquals(pkg.Dir, dir) {
 			continue
 		}
 		for _, f := range pkg.Files {

context/rewrite.go

@@ -101,13 +101,13 @@ func (ctx *Context) rewrite() error {
 		Tabwidth: 8,
 	}
 	for _, fileInfo := range filePaths {
-		if pathos.FileHasPrefix(fileInfo.Path, ctx.RootDir) == false {
+		if !pathos.FileHasPrefix(fileInfo.Path, ctx.RootDir) {
 			continue
 		}
 
 		// Read the file into AST, modify the AST.
 		fileset := token.NewFileSet()
-		f, err := parser.ParseFile(fileset, fileInfo.Path, nil, parser.ParseComments)
+		f, _ := parser.ParseFile(fileset, fileInfo.Path, nil, parser.ParseComments)
 		if f == nil {
 			return nil
 		}

context/version.go

@@ -29,7 +29,7 @@ func isVersion(s string) bool {
 	if hasPunct {
 		return true
 	}
-	if onlyHexLetter == false {
+	if !onlyHexLetter {
 		return true
 	}
 

migrate/godep.go

@@ -93,7 +93,7 @@ func (sysGodep) Migrate(root string) error {
 
 	for _, d := range godeps.Deps {
 		for _, pkg := range ctx.Package {
-			if strings.HasPrefix(pkg.Path, d.ImportPath) == false {
+			if !strings.HasPrefix(pkg.Path, d.ImportPath) {
 				continue
 			}
 			vf := ctx.VendorFilePackagePath(pkg.Path)

migrate/migrate.go

@@ -112,7 +112,7 @@ func hasDirs(root string, dd ...string) bool {
 		if err != nil {
 			return false
 		}
-		if fi.IsDir() == false {
+		if !fi.IsDir() {
 			return false
 		}
 	}
@@ -125,7 +125,7 @@ func hasFiles(root string, dd ...string) bool {
 		if err != nil {
 			return false
 		}
-		if fi.IsDir() == true {
+		if fi.IsDir() {
 			return false
 		}
 	}

run/filter.go

@@ -164,7 +164,7 @@ func insertListToAllNot(sg *context.StatusGroup, list []context.Status) {
 	if len(sg.Group) == 0 {
 		allStatusNot := true
 		for _, s := range sg.Status {
-			if s.Not == false {
+			if !s.Not {
 				allStatusNot = false
 				break
 			}

run/license.go

@@ -87,7 +87,7 @@ func (r *runner) License(w io.Writer, subCmdArgs []string) (help.HelpMessage, er
 	}
 
 	for _, item := range list {
-		if f.HasStatus(item) == false {
+		if !f.HasStatus(item) {
 			continue
 		}
 		if len(f.Import) != 0 && f.FindImport(item) == nil {

run/list.go

@@ -99,7 +99,7 @@ func (r *runner) List(w io.Writer, subCmdArgs []string) (help.HelpMessage, error
 	tw := tabwriter.NewWriter(w, 0, 4, 2, ' ', 0)
 	defer tw.Flush()
 	for _, item := range list {
-		if f.HasStatus(item) == false {
+		if !f.HasStatus(item) {
 			continue
 		}
 		if len(f.Import) != 0 && f.FindImport(item) == nil {

vcs/bzr.go

@@ -23,7 +23,7 @@ func (VcsBzr) Find(dir string) (*VcsInfo, error) {
 		}
 		return nil, err
 	}
-	if fi.IsDir() == false {
+	if !fi.IsDir() {
 		return nil, nil
 	}
 

vcs/git.go

@@ -23,7 +23,7 @@ func (VcsGit) Find(dir string) (*VcsInfo, error) {
 		}
 		return nil, err
 	}
-	if fi.IsDir() == false {
+	if !fi.IsDir() {
 		return nil, nil
 	}
 

vcs/hg.go

@@ -23,7 +23,7 @@ func (VcsHg) Find(dir string) (*VcsInfo, error) {
 		}
 		return nil, err
 	}
-	if fi.IsDir() == false {
+	if !fi.IsDir() {
 		return nil, nil
 	}
 

vcs/svn.go

@@ -23,7 +23,7 @@ func (svn VcsSvn) Find(dir string) (*VcsInfo, error) {
 		}
 		return nil, err
 	}
-	if fi.IsDir() == false {
+	if !fi.IsDir() {
 		return nil, nil
 	}
 

vcs/vcs.go

@@ -70,7 +70,7 @@ func FindVcs(root, packageDir string) (info *VcsInfo, err error) {
 		if nextPath == path {
 			return nil, nil
 		}
-		if pathos.FileHasPrefix(nextPath, root) == false {
+		if !pathos.FileHasPrefix(nextPath, root) {
 			return nil, nil
 		}
 		path = nextPath

vendorfile/file.go

@@ -139,7 +139,7 @@ loop:
 				continue loop
 			}
 			*field = value
-			if value == true {
+			if value {
 				break loop
 			}
 		}
@@ -160,7 +160,7 @@ func setObject(fieldObj interface{}, object map[string]interface{}, names []stri
 		}
 	case bool:
 		for i, name := range names {
-			if i != 0 || (hideEmpty && field == false) {
+			if i != 0 || (hideEmpty && !field) {
 				delete(object, name)
 				continue
 			}