fix: do not abort check on ignored fields (#88)

Author: 22dm

musttag.go

@@ -222,7 +222,7 @@ func (c *checker) checkStruct(styp *types.Struct, tag string) (valid bool) {
 
 		// Do not recurse into ignored fields.
 		if tagValue == "-" {
-			return true
+			continue
 		}
 
 		if valid := c.checkType(field.Type(), tag); !valid {

testdata/src/tests/tests.go

@@ -165,6 +165,22 @@ func ignoredNestedType() {
 	json.Marshal(&Foo{}) // no error
 }
 
+func ignoredNestedTypeWithSubsequentNoTagField() {
+	type Nested struct {
+		NoTag string
+	}
+	type Foo struct {
+		Ignored  Nested `json:"-"`
+		Exported string `json:"exported"`
+		NoTag    string
+	}
+	var foo Foo
+	json.Marshal(foo)    // want "the given struct should be annotated with the `json` tag"
+	json.Marshal(&foo)   // want "the given struct should be annotated with the `json` tag"
+	json.Marshal(Foo{})  // want "the given struct should be annotated with the `json` tag"
+	json.Marshal(&Foo{}) // want "the given struct should be annotated with the `json` tag"
+}
+
 func interfaceSliceType() {
 	type WithMarshallableSlice struct {
 		List []Marshaler `json:"marshallable"`