mtkennerly / ludusavi-manifest

Backup info for PC game saves
MIT License
74 stars 13 forks source link

cannot parse `manifest.yml` with `yq` #49

Closed aanderse closed 2 months ago

aanderse commented 2 months ago

i can't seem to parse manifest.yaml with yq because an invalid character

i have tried several different python libraries and all fail to parse the file

any hints?

$ yq '.' manifest.yaml
yq: Error running jq: ReaderError: unacceptable character #x0081: control characters are not allowed
  in "manifest.yaml", position 1452746.

any and all help appreciated :bowing_man:

mtkennerly commented 2 months ago

Hi! It looks like serde_yaml 0.8.26 doesn't properly escape control characters in strings. I'm avoiding upgrading that dependency for now, so I've added a filter for those characters instead. yq seems to be happy now :)

aanderse commented 2 months ago

great! thank you very much! ❤️