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

[Clippy] Code maintenance #165

Merged
merged 1 commit into from
Oct 20, 2024
Merged

[Clippy] Code maintenance #165

merged 1 commit into from
Oct 20, 2024

Conversation

AzHicham
Copy link
Owner

🤖 This is an automatic PR 🤖
Clippy suggests the following changes

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

philips-isyntax-rs Benchmark

Benchmark suite Current: 6f9aee3 Previous: 132c667 Ratio
philips_i2syntax_read_region_256_lvl_0 1350325 ns/iter (± 79765) 1382074 ns/iter (± 101204) 0.98
philips_i2syntax_read_region_256_lvl_1 1148009 ns/iter (± 51301) 1177604 ns/iter (± 50886) 0.97
philips_i2syntax_read_region_512_lvl_0 1542212 ns/iter (± 44034) 1682424 ns/iter (± 71560) 0.92
philips_i2syntax_read_region_512_lvl_1 7374102 ns/iter (± 326816) 7612744 ns/iter (± 299313) 0.97
philips_read_region_256_lvl_0 513251 ns/iter (± 39814) 533197 ns/iter (± 45958) 0.96
philips_read_region_256_lvl_1 1228251 ns/iter (± 48865) 1264376 ns/iter (± 68238) 0.97
philips_read_region_512_lvl_0 3518946 ns/iter (± 251689) 3567126 ns/iter (± 289753) 0.99
philips_read_region_512_lvl_1 2730458 ns/iter (± 83244) 2757279 ns/iter (± 93814) 0.99

This comment was automatically generated by workflow using github-action-benchmark.

Copy link

codecov bot commented Oct 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.02%. Comparing base (5e3f7c7) to head (6f9aee3).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #165   +/-   ##
=======================================
  Coverage   91.02%   91.02%           
=======================================
  Files           8        8           
  Lines         568      568           
=======================================
  Hits          517      517           
  Misses         51       51           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AzHicham AzHicham merged commit d06dd4b into main Oct 20, 2024
10 checks passed
@AzHicham AzHicham deleted the auto/clippy branch October 20, 2024 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants