yq merge panics if comments contains Unicode

[lingsamuel]

Describe the bug
When the comment contains Unicode such as Chinese, yq merge will panic.

Input Yaml
Minimal test yamls below:

a.yaml:

a: work

b.yaml:

中文: work

c.yaml:

c: error # 中文

d.yaml

d: 中文

Command

yq merge a.yaml b.yaml # Okay
yq merge a.yaml c.yaml # Panic!
yq merge a.yaml d.yaml # Okay
yq merge b.yaml d.yaml # Okay

Actual behavior

From the stack trace, maybe it's not a yq bug but a yaml.v3 bug.

Expected behavior

Comments with Unicode should work.

Additional context

yq version:

[lingsamuel]

yaml.v3 bug: go-yaml/yaml#538
It's closed but still exists...

[mikefarah]

Yeah I'm hoping someone on that project will pick it up :(

[lingsamuel]

go-yaml/yaml#538 (comment)
I think the fix will fly soon.

[lingsamuel]

Hello, yaml.v3 fixed this issue: go-yaml/yaml#538 (comment).

[mikefarah]

yup - next release will have that fix!