Closed strangelookingnerd closed 3 months ago
Failure in Lint / PHP-CS-Fixer action seems to be unrelated to my changes.
Please note that all changes should be based on and target development-v6
as we do not plan to add new features to v5
Please note that all changes should be based on and target
development-v6
as we do not plan to add new features to v5
Oh, I see. Will re-create the PR for a clean history. This bit of info should be added to the contributing section as is still states devel
.
Thank you for your contribution to the Pi-hole Community!
Please read the comments below to help us consider your Pull Request.
We are all volunteers and completing the process outlined will help us review your commits quicker.
Please make sure you
What does this PR aim to accomplish?:
CPU load in the header is currently rounded which can result in the value being 0 or 1 rather than 0.00 and 1.00. Adding the additional zeros in my opinion gives a better visual appearance.
How does this PR accomplish the above?:
Replace
round
withnumber_format
and also making sure to use a dot as decimal separator. Sincenumber_format
is taking the current locale into account this could otherwise lead to the separator being a comma.Link documentation PRs if any are needed to support this PR:
By submitting this pull request, I confirm the following:
git rebase
)