Skip to content

Conversation

@aastopher
Copy link
Owner

  • resolve clippy warnings.
  • remove wildcard imports (left 1 case in tests).
  • shift select_algo and select_filter_type to imgddcore reducing redundant code.

@deepsource-io
Copy link

deepsource-io bot commented Feb 11, 2025

Here's the code health analysis summary for commits 56600a6..40d7641. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Rust LogoRust✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 11, 2025

CodSpeed Performance Report

Merging #29 will not alter performance

Comparing feature/clippy (40d7641) with stage (8ddfd96)

Summary

✅ 10 untouched benchmarks

@aastopher aastopher changed the base branch from main to stage February 11, 2025 04:00
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (4507bd3) to head (40d7641).
Report is 87 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff            @@
##             stage       #29   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         6    +1     
  Lines          186       172   -14     
=========================================
- Hits           186       172   -14     

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

@aastopher aastopher merged commit 4ce5e99 into stage Feb 11, 2025
10 checks passed
@aastopher aastopher deleted the feature/clippy branch February 11, 2025 04: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.

3 participants