Skip to content

feat(cli): add --verbose to export quiet logging#933

Open
Solaris-star wants to merge 2 commits into
repowise-dev:mainfrom
Solaris-star:feat/export-verbose-logging
Open

feat(cli): add --verbose to export quiet logging#933
Solaris-star wants to merge 2 commits into
repowise-dev:mainfrom
Solaris-star:feat/export-verbose-logging

Conversation

@Solaris-star

Copy link
Copy Markdown

Summary

  • Add --verbose / -v to repowise export, wired to configure_cli_logging so progress bars stay clean by default (same pattern as init/update/reindex).
  • Unit test asserts the flag reaches configure_cli_logging before pipeline work.

Part of #930 (export slice only).

How tested

.venv/bin/python -m pytest tests/unit/cli/test_export_cmd.py -q
# 1 passed

Wire export to configure_cli_logging so progress bars stay clean by
default, matching init/update/reindex. Part of repowise-dev#930.

Signed-off-by: Solaris-star <820622658@qq.com>
@repowise-bot

repowise-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

✅ Health: 7.5 (unchanged)

📋 At a glance
1 hotspot touched · 1 new finding introduced. Scoped to packages.

🗺️ Change map

flowchart LR
  subgraph PR ["Changed in this PR (1 with dependents)"]
    f_packages_cli_src_repowise_cli_commands_export_cmd_py[".../commands/export_cmd.py 🔥"]:::changed
  end
  f_packages_cli_src_repowise_cli_main_py[".../cli/main.py"]
  f_packages_cli_src_repowise_cli_commands_export_cmd_py --> f_packages_cli_src_repowise_cli_main_py
  classDef changed fill:#dbeafe,stroke:#1d4ed8,color:#1e3a5f
  classDef warn fill:#fef3c7,stroke:#b45309,color:#78350f
  classDef guard fill:#dcfce7,stroke:#15803d,color:#14532d
Loading

Solid arrows: code that imports the changed files (1 direct dependent, from the last indexed snapshot). Dashed: history/tests.

🔎 More signals (1)

🔥 Hotspot touched (1)

  • .../commands/export_cmd.py — 0 commits/90d, 1 dependents · primary owner: Raghav Chamadiya (100%)

📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-24 01:08 UTC
Silence on a single PR with [skip repowise] in the title · Per-repo toggle on repowise.dev/settings?tab=bot

@RaghavChamadiya RaghavChamadiya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Keeping this one since you claimed the slice first. Same three items as #932: route the test through CliRunner rather than .callback(), add the flag to the repowise export options table in docs/CLI_REFERENCE.md (around line 657), and run ruff format . (it reformats both files here).

@Narasimha2211 opened #934 for the same command with a test that already does the CliRunner version and covers the quiet default. Please lift that test shape and credit them in the commit body.

Route the verbose-flag test through CliRunner().invoke() instead of
.callback() so it actually exercises the Click decorator (repowise-dev#933).
Add --verbose/-v to the export options table in CLI_REFERENCE.md.

Co-authored-by: Narasimha2211 (test shape from repowise-dev#934)
@Solaris-star

Copy link
Copy Markdown
Author

Same three items addressed:

  1. Test now uses CliRunner().invoke(cli, ["export", ..., "-v"]) instead of .callback()
  2. Added --verbose/-v to the export options table in docs/CLI_REFERENCE.md
  3. Ran ruff format

Lifted the test shape from #934 (credited in commit body). Please re-review when convenient.

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