Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🤖 use clechasseur/rs-clippy-check@v3 for clippy PR reviews #318

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

ttytm
Copy link
Owner

@ttytm ttytm commented Jun 11, 2024

resolves deprecation warnings

Copy link

Filename                              Stmts    Miss  Cover    Missing
----------------------------------  -------  ------  -------  ------------------------------------------------------------
src/main.rs                              13      13  0.00%    26-42
src/modules/api.rs                       24      13  45.83%   26-29, 38, 49-88
src/modules/args.rs                       4       4  0.00%    91-95
src/modules/config.rs                    31      25  19.35%   52-97
src/modules/forecast.rs                  20       6  70.00%   7-8, 24-25, 27-28
src/modules/localization.rs             104      40  61.54%   162-203, 230-262
src/modules/location.rs                  55      28  49.09%   25-29, 59-63, 75, 89, 95-104, 109, 118-126, 131-132, 142-143
src/modules/params.rs                    55      55  0.00%    24-137
src/modules/units.rs                     13       0  100.00%
src/modules/weather.rs                   26      26  0.00%    83-154
src/modules/display/border.rs            47      47  0.00%    17-106
src/modules/display/current.rs           83      83  0.00%    37-228
src/modules/display/day.rs               57      57  0.00%    29-164
src/modules/display/graph.rs            171     166  2.92%    65-368
src/modules/display/gui_config.rs        15      12  20.00%   46-61
src/modules/display/historical.rs        61      61  0.00%    35-201
src/modules/display/hourly.rs           193     193  0.00%    40-396
src/modules/display/product.rs           19      19  0.00%    24-64
src/modules/display/utils.rs             46      46  0.00%    6-85
src/modules/display/weathercode.rs       31      31  0.00%    11-41
src/modules/display/week.rs              51      51  0.00%    30-137
src/modules/display/wind.rs              21      21  0.00%    17-42
TOTAL                                  1140     997  12.54%

@ttytm ttytm force-pushed the ci/update-clippy-check branch from 2193d58 to f5f297b Compare June 11, 2024 16:34
@ttytm ttytm changed the title 🤖 use actions-rs-plus/clippy-check@v2 for clippy PR reviews 🤖 use clechasseur/rs-clippy-check@v3 for clippy PR reviews Jun 11, 2024
@ttytm ttytm merged commit 1dd6de5 into main Jun 11, 2024
11 checks passed
@ttytm ttytm deleted the ci/update-clippy-check branch June 11, 2024 16:39
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.

1 participant