Add nolint with reasons to ignore unwanted lint issues

yos/compare.go

@@ -70,6 +70,7 @@ func SameFileContent(path1, path2 string) (same bool, err error) {
 }
 
 // SameDirEntries checks if the two directories have the same entries. Symbolic links other than the given paths will be not be followed, and only compares content of links.
+//nolint:gocyclo // Checks in this function are all necessary, no redundant checks.
 func SameDirEntries(path1, path2 string) (same bool, err error) {
 	var (
 		fi1, fi2       os.FileInfo

yos/copy.go

@@ -58,6 +58,7 @@ func CopySymlink(src, dest string) (err error) {
 }
 
 // bufferCopyFile reads content from the source file and write to the destination file with a buffer.
+//nolint:gocyclo // buffer copy is a complicated thing indeed.
 func bufferCopyFile(src, dest string, bufferSize int64) (err error) {
 	var (
 		srcFile, destFile *os.File
@@ -161,6 +162,7 @@ func copySymlink(src, dest string) (err error) {
 }
 
 // copyDir copies all entries of source directory to destination directory recursively.
+//nolint:gocyclo // copy directory refers itself with copy file and copy symlink, it's hard to reduce the complexity.
 func copyDir(src, dest string) (err error) {
 	var srcInfo, destInfo os.FileInfo
 

yrand/common.go

@@ -10,6 +10,7 @@ import (
 var (
 	// QuitShuffle is used as a return value from ShuffleIndexFunc to indicate that the execution of WeightedShuffle should be terminated immediately.
 	// It is not returned as an error by any function.
+	//nolint:golint // this is not a real error returned by any functions, it works as a flag instead.
 	QuitShuffle = errors.New("quit this shuffle")
 )