Various small bugfixes - Githubissues

HorlogeSkynet / archey4

:computer: Maintained fork of the original Archey (Linux) system tool

https://git.io/archey4

GNU General Public License v3.0

291 stars 37 forks source link

Various small bugfixes #117

Closed ingrinder closed 2 years ago

ingrinder commented 2 years ago

Description

A few commits that fix some small bugs I've found in Archey.

Reason and / or context

Fixes the following issues:

[Model] Prevents the vendor name being repeated when the product name already includes the vendor name, as on HP Laptops
[Window Manager] Adds the Wayland variant of KWin to the process list
[Load Average] Cuts off the load averages to 2 decimal places (configurable) in line with other tools like uptime

How has this been tested ?

Tested on my systems and passes unit tests added & written

Types of changes :

[X] Bug fix (non-breaking change which fixes an issue)
Typo / style fix (non-breaking change which improves readability)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist :

[X] [IF NEEDED] I have updated the README.md file accordingly ;
[X] [IF NEEDED] I have updated the test cases (which pass) accordingly ;
[IF BREAKING] This pull request targets next Archey version branch ;
[X] My changes looks good ;
[X] I agree that my code may be modified in the future ;
[X] My code follows the code style of this project (PEP8).

HorlogeSkynet commented 2 years ago

Many thanks @ingrinder, I'll give a proper review/propose you some changes within some days 🙏

ingrinder commented 2 years ago

Apologies for the delay - those changes LGTM, feel free to merge/rebase :+1:

(sorry for not squashing the tests into the commits - should have thought of that!)