Thanks to visit codestin.com
Credit goes to github.com

Skip to content

refactor: Address diverse clippy findings#1799

Merged
gtema merged 1 commit into
mainfrom
clippy
Jun 18, 2026
Merged

refactor: Address diverse clippy findings#1799
gtema merged 1 commit into
mainfrom
clippy

Conversation

@gtema

@gtema gtema commented Jun 18, 2026

Copy link
Copy Markdown
Owner
  • Box large enum variants in Action, TuiError, CloudWorkerError, and
    IdentityCommands to reduce stack size and eliminate large_enum_variant
    warnings; add manual From impls for boxed error types
  • Eliminate unnecessary clones: replace .clone() with as_deref(),
    as_str(), and moves in render loops (header, fuzzy_select_list,
    table_view)
  • Replace .to_owned() with .clone() in normalized_filters
    (implicit_clone)
  • Use .clone_from() instead of assignment + .clone() (assigning_clones)
  • Replace .iter_mut() loops with &mut (explicit_iter_loop)
  • Remove redundant closure in table_view .map(|item| item.status())
  • Move sources directly in config::ConfigFileBuilder::build() instead of
    borrow + clone
  • auth-token: remove unnecessary borrow in _get_auth_data call

- Box large enum variants in Action, TuiError, CloudWorkerError, and
  IdentityCommands to reduce stack size and eliminate large_enum_variant
  warnings; add manual From impls for boxed error types
- Eliminate unnecessary clones: replace .clone() with as_deref(),
  as_str(), and moves in render loops (header, fuzzy_select_list,
  table_view)
- Replace .to_owned() with .clone() in normalized_filters
  (implicit_clone)
- Use .clone_from() instead of assignment + .clone() (assigning_clones)
- Replace .iter_mut() loops with &mut (explicit_iter_loop)
- Remove redundant closure in table_view .map(|item| item.status())
- Move sources directly in config::ConfigFileBuilder::build() instead of
  borrow + clone
- auth-token: remove unnecessary borrow in _get_auth_data call
@gtema gtema changed the title refactor: Address clippy issues refactor: Address diverse clippy findings Jun 18, 2026
@gtema gtema merged commit 5549480 into main Jun 18, 2026
20 checks passed
@gtema-release-plz gtema-release-plz Bot mentioned this pull request Jun 18, 2026
@gtema gtema deleted the clippy branch June 28, 2026 07:26
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