Enable more linters

.golangci.yml

@@ -41,16 +41,17 @@ linters-settings:
 linters:
   disable-all: true
   enable:
+    - bidichk
     - errcheck
     - goconst
     - gocyclo
-    # - gofmt       # handled by safer-golangci-lint.yml
-    # - goimports   # handled by safer-golangci-lint.yml
+    - gofmt
+    - goimports
     - gosec
     - govet
     - ineffassign
     - misspell
-    # - revive      # temporarily disabled to reduce noise in golangci-lint 1.52.2
+    - revive
     - staticcheck
     - typecheck
     - unconvert

encode.go

@@ -1266,11 +1266,7 @@ func encodeTag(e *encoderBuffer, em *encMode, v reflect.Value) error {
 	encodeHead(e, byte(cborTypeTag), t.Number)
 
 	// Marshal tag content
-	if err := encode(e, em, reflect.ValueOf(t.Content)); err != nil {
-		return err
-	}
-
-	return nil
+	return encode(e, em, reflect.ValueOf(t.Content))
 }
 
 func encodeSimpleValue(e *encoderBuffer, em *encMode, v reflect.Value) error {

simplevalue.go

@@ -4,10 +4,11 @@ import "reflect"
 
 // SimpleValue represents CBOR simple value.
 // CBOR simple value is:
-// * an extension point like CBOR tag.
-// * a subset of CBOR major type 7 that isn't floating-point.
-// * "identified by a number between 0 and 255, but distinct from that number itself".
-//   For example, "a simple value 2 is not equivalent to an integer 2" as a CBOR map key.
+//   - an extension point like CBOR tag.
+//   - a subset of CBOR major type 7 that isn't floating-point.
+//   - "identified by a number between 0 and 255, but distinct from that number itself".
+//     For example, "a simple value 2 is not equivalent to an integer 2" as a CBOR map key.
+//
 // CBOR simple values identified by 20..23 are: "false", "true" , "null", and "undefined".
 // Other CBOR simple values are currently unassigned/reserved by IANA.
 type SimpleValue uint8

structfields.go

@@ -129,7 +129,7 @@ func getFields(t reflect.Type) (flds fields, structOptions string) {
 		}
 
 		// Skip fields with the same field name.
-		for i++; i < len(flds) && name == flds[i].name; i++ {
+		for i++; i < len(flds) && name == flds[i].name; i++ { //nolint:revive
 		}
 	}
 	if j != len(flds) {