ad-oliviero / uwufetch

A meme system info tool for Linux, based on nyan/uwu trend on r/linuxmasterrace.
GNU General Public License v3.0
748 stars 52 forks source link

Why use json to a config file? #83

Closed Guaxinim5573 closed 3 years ago

Guaxinim5573 commented 3 years ago

This is a discussion, not an actual issue.

Recently, UwUFetch started using JSON for the config file using an external library, but I think JSON isn't good for a config file a human would write. Here is why:

  1. JSON isn't very readable for humans.
  2. It can cause more syntax errors.
  3. A config file should be easy to understand and write, without many rules.
  4. Need an external library

I think the old approach using a simple KEY=VALUE was better than a JSON file. Here is why:

  1. More readable for humans.
  2. It's simple to write, so there are not many syntax errors.
  3. A custom parser isn't hard to make, no need to use an external library.

EDIT: Spelling errors.

ad-oliviero commented 3 years ago

The problems caused by the external library are enough to revert, I'm rebuilding the config parser.

ad-oliviero commented 3 years ago

Now using custom parser (09b32b1), I'll soon publish a new release

ad-oliviero commented 3 years ago

I just released 1.4, so this issue and #82 are fixed