fix formatting of nolint tags

Also replacing some of them, as there's more linters that check for this
now (revive being one of them), so "fixing" them prevents having to add
more linters to the list.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

assert/cmd/gty-migrate-from-testify/call.go

@@ -22,8 +22,7 @@ type call struct {
 
 func (c call) String() string {
 	buf := new(bytes.Buffer)
-	//nolint: errcheck
-	format.Node(buf, token.NewFileSet(), c.expr)
+	_ = format.Node(buf, token.NewFileSet(), c.expr)
 	return buf.String()
 }
 

assert/cmp/compare.go

@@ -249,7 +249,7 @@ type causer interface {
 }
 
 func formatErrorMessage(err error) string {
-	//nolint: errorlint // unwrapping is not appropriate here
+	//nolint:errorlint // unwrapping is not appropriate here
 	if _, ok := err.(causer); ok {
 		return fmt.Sprintf("%q\n%+v", err, err)
 	}

fs/file.go

@@ -72,8 +72,7 @@ func (f *File) Path() string {
 
 // Remove the file
 func (f *File) Remove() {
-	//nolint: errcheck
-	os.Remove(f.path)
+	_ = os.Remove(f.path)
 }
 
 // Dir is a temporary directory
@@ -106,8 +105,7 @@ func (d *Dir) Path() string {
 
 // Remove the directory
 func (d *Dir) Remove() {
-	//nolint: errcheck
-	os.RemoveAll(d.path)
+	_ = os.RemoveAll(d.path)
 }
 
 // Join returns a new path with this directory as the base of the path