API to change error behavior about go-yaml HOT 7 CLOSED

@goccy ちょっとJIRA業を逃れたくてPoC書いてみました lestrrat-go@d6a5e04

from go-yaml.

ありがとうございます！
たしかに今の設計ですと yaml パッケージのみで完結していた方が綺麗ですね。

yaml パッケージの API に対して依頼を投げる形で同意なのですが、
yaml.PrettyPrintError() 内で標準出力に書き出してしまっていいかは少し考えてみたいです。

使い方によっては、 Unmarshal の err をラップして使うような方もいるのかなと思っていたのですが、もしそういった用途を想定するなら、書き出しタイミングや内容を制御できる方が良いのかなとも考えました。

from go-yaml.

@lestrrat
なるほど、 yaml.FormatError 内で color / source コード付きの制御をした後にその結果を string として返してくれるんですね。この設計ならもともとの意図通りですし、何より綺麗です...！

LGTM です！

from go-yaml.

@goccy おす。じゃあ清書してPRしますね

from go-yaml.

@goccy あ、ただ、デフォルトの挙動が逆になります（err.Error()の戻り値のデフォルトはcolored=false,inclSource=false）これでもいいです？

from go-yaml.

今やってみたけど、colored=false, inclSource=trueがデフォルトがいいような気がするな。ともあれ、いずれにせよご検討ください。

from go-yaml.

@lestrrat 僕も colored=false, inclSource=true がデフォのほうがいいような気がします...！

from go-yaml.