dualsensectl icon indicating copy to clipboard operation
dualsensectl copied to clipboard

feat: Add JSON output option to battery and device list

Open ladydascalie opened this issue 2 years ago • 2 comments

This is a first pass investigating how to add JSON output to some (or all?) of the commands. As I am not an experienced C programmer, any feedback is welcome, however for reference my plan was to simply use printf and manually handle the output in order to avoid having to bring in any third party dependencies for json serialization.

As there isn't a huge amount of output, I believe this is sustainable for the time being, but easily reworked if need be.

ladydascalie avatar Nov 12 '23 18:11 ladydascalie

Addresses #26 (my own issue) in order to serve as a base for deciding on an approach

ladydascalie avatar Nov 12 '23 18:11 ladydascalie

I think it would be better to just do a global flag instead of adding it to each command: dualsensectl -j battery.

nowrep avatar Nov 13 '23 12:11 nowrep

I honestly just don't have the time to look at this at the moment, so unfortunately I'm closing this. If anyone wants to take it over the finish line, feel free, but I don't want to leave this open forever when I'm not able to work on it.

ladydascalie avatar Jun 11 '24 18:06 ladydascalie