Inconsistent struct tag use

[lestrrat]

I guess the intent of this is to allow

type T struct {
    A `foo`
}

to just work, but I don't think this is a common practice (at least, encoding/json doesn't do that), and I also think it's a bad idea.

At least, the current code does not check for this:

type T struct {
    A `"foo"` // does NOT do what we want
}

of course, it's easy to handle this particular case, but it's just one more inconsistency. I think it's much much better to say "we support tag name X (and Y)" and be done with it.

[goccy]

I implemented this behavior with reference to go-yaml/yaml ,
but I'm also wondering if this feature is needed, so I consider removing it .

• https://github.com/go-yaml/yaml/blob/v2/decode_test.go#L253-L258
• https://github.com/go-yaml/yaml/blob/v2/yaml.go#L331-L333

[lestrrat]

if the goal is to be compatible with other libraries, as long as there's a big fat warning in the code and docs, I think it's okay.

if it's up for a toss, I would say it's not necessary

[goccy]

I fixed this issue with #44