Skip to content

Fix rounding in precipitation percentage#3045

Merged
khassel merged 5 commits intoMagicMirrorOrg:developfrom
rejas:bugfix/pop_round
Feb 20, 2023
Merged

Fix rounding in precipitation percentage#3045
khassel merged 5 commits intoMagicMirrorOrg:developfrom
rejas:bugfix/pop_round

Conversation

@rejas
Copy link
Collaborator

@rejas rejas commented Feb 19, 2023

Percentage should be always rounded so that we dont get something like "47.0000000001 %"

Some small typo and naming fixes also while I am here

@codecov-commenter
Copy link

Codecov Report

Merging #3045 (e670a0b) into develop (bf28e63) will increase coverage by 0.07%.
The diff coverage is 57.89%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##           develop    #3045      +/-   ##
===========================================
+ Coverage    22.91%   22.98%   +0.07%     
===========================================
  Files           52       52              
  Lines        11543    11545       +2     
===========================================
+ Hits          2645     2654       +9     
+ Misses        8898     8891       -7     
Impacted Files Coverage Δ
...odules/default/weather/providers/openweathermap.js 0.00% <0.00%> (ø)
js/server_functions.js 91.33% <100.00%> (ø)
modules/default/utils.js 39.45% <100.00%> (ø)
modules/default/weather/weatherutils.js 99.30% <100.00%> (ø)
modules/default/updatenotification/node_helper.js 90.27% <0.00%> (+12.50%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@khassel khassel merged commit 6d86ffa into MagicMirrorOrg:develop Feb 20, 2023
@rejas rejas deleted the bugfix/pop_round branch February 20, 2023 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants