Skip to content

fix: remove redundant printing#337

Open
Cyber-Syntax wants to merge 2 commits into
mainfrom
fix/update-all-wrong-print
Open

fix: remove redundant printing#337
Cyber-Syntax wants to merge 2 commits into
mainfrom
fix/update-all-wrong-print

Conversation

@Cyber-Syntax
Copy link
Copy Markdown
Owner

Problem

Redundant logging during the update command was cluttering the console output, making it difficult for users to focus on important information.

Solution

Removed the unnecessary logging statements that printed the installation status of applications during the update process. This change simplifies the output and enhances user experience.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation
  • Other (please describe)

Checklist

  • If you changed python module:
    • Run all fast tests: uv run pytest -m 'not slow'
    • Run e2e tests: uv run pytest tests/e2e/ (WARNING: This tests would use real api connection, make sure you setup your token. Also, this would take time according to your internet speed.)
  • Add your test (if you can)
  • Add/update docs

Fixes #233

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 4 files

Tip: cubic could auto-approve low-risk PRs like this, if it thinks it's safe to merge. Learn more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready

Development

Successfully merging this pull request may close these issues.

🐞 bug: update all command wrong ui

1 participant