No support for alacritty TOML config

hykilpikonna / hyfetch

🏳️‍🌈 🏳️‍⚧️ Neofetch with LGBTQ+ pride flags!

MIT License

1.18k stars 93 forks source link

No support for alacritty TOML config #200

Closed LyricLy closed 9 months ago

LyricLy commented 9 months ago

Describe the bug

alacritty has changed config format from YAML to TOML. hyfetch is only able to parse the old alacritty.yml files, so it fails to correctly report the font in use.

[x] Does this issue still occur in the master branch? (Required)

Expected behavior

For my terminal font to be reported in hyfetch's output.

Screenshots

It's simply not there.

BKasin commented 9 months ago

Does this match what you would expect when the config was YAML?

alacritty

hykilpikonna commented 9 months ago

This is fixed by #202 (Thanks to @BKasin!)

LyricLy commented 8 months ago

It looks as though the detection is the wrong way around? It recognizes .yml files as TOML and .toml files as YAML.

BKasin commented 8 months ago

The detection was correct, the awk logic was reversed. So it would properly detect a TOML file and then parse it with YAML logic. I think what happened was I had it set up properly, and then last second decided to switch to check for TOML first and didn't actually finish making that switch and got distracted. But it should be fixed in 42a275b27bc4689d76d7c8f947478f776b351bd9. Sorry about that.