Improved battery detection

[jan-guenter]

Improvements to the battery detection:

• now considering all power supplies (instead of path name filtering)
• only consider power supplies that are currently present
• only consider power supplies of type Batter or UPS

All changes made are in accordants to the definitions of /sys/class/power_supply from the kernel docs

---

This change was made to address the battery issue part of issue #94

This issue is still draft status, since I'm still awaiting feedback by the issue reporter @bionade24 on the test build I provided.

[aristocratos]

Looks good, thanks for the fix.

[aristocratos]

@jan-guenter
Isn't the path 'src/**' recursive for the workflow trigger? Noticed it didn't trigger when merging this.

[jan-guenter]

@aristocratos

Isn't the path 'src/**' recursive for the workflow trigger? Noticed it didn't trigger when merging this.

According to the docs it should https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet.
This actually is my first time working with GitHub Actions myself, so I'm not too familiar with them, but don't see why it didn't trigger. Perhaps a good time to use the GitHub support you are paying for 😄
Please let me know if you find out why it didn't trigger.

[jan-guenter]

@aristocratos
Perhaps the workflow_dispatch trigger I added to allow manual triggers interferes somehow, but I cannot find a good reason for it in the docs.
I also have workflows in my fork that have workflow_dispatch as well as push triggers and they work fine.
https://github.com/jan-guenter/btop/blob/dcfd786582e9389da778b6746c79f9d7d2cf6f63/.github/workflows/cmake-builds.yml#L3-L7

[aristocratos]

@jan-guenter
Found it, you changed the targeted branch from "main" to "master" when you updated the yml :P

[jan-guenter]

Found it, you changed the targeted branch from "main" to "master" when you updated the yml :P

ouch... sorry 😅