Skip to content

Conversation

@GeekMasher
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 14, 2025 18:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CLI interface and the logging output while introducing a minor change in the interactive mode selection flow.

  • Updated logging output by incorporating the console style for colored logging messages.
  • Modified the select_mode function signature and updated dialoguer usage for fuzzy selection in interactive mode.
  • Updated dependency configuration for dialoguer to enable fuzzy selections.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

File Description
src/main.rs Added console styling for version output and updated select_mode call.
src/interactive.rs Changed select_mode to accept a Config parameter and switched from Select to FuzzySelect.
src/cli.rs Added custom logging formatting with colored level indicators.
Cargo.toml Updated dialoguer dependency to include the fuzzy-select feature.
Comments suppressed due to low confidence (1)

src/interactive.rs:10

  • [nitpick] Consider using 'config.version.is_none()' instead of '!config.version.is_some()' for improved clarity and readability.
if !config.version.is_some() {

let selection = Select::new()
pub fn select_mode(config: &Config) -> Result<WorkflowMode> {
let mut modes = Vec::new();
if !config.version.is_some() {

Check warning

Code scanning / clippy

this boolean expression can be simplified Warning

this boolean expression can be simplified
let selection = Select::new()
pub fn select_mode(config: &Config) -> Result<WorkflowMode> {
let mut modes = Vec::new();
if !config.version.is_some() {

Check warning

Code scanning / clippy

this boolean expression can be simplified Warning

this boolean expression can be simplified
@GeekMasher GeekMasher enabled auto-merge (rebase) April 14, 2025 18:50
@GeekMasher GeekMasher merged commit fef256b into main Apr 14, 2025
11 checks passed
@GeekMasher GeekMasher deleted the feat/interface-updates branch April 14, 2025 18:50
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.

2 participants