Properly handle when all packages are up to date

[danifr]

No description provided.

[simion]

AI review - Claude

🟢 Good (2)

• Correct replacement of KeyboardInterrupt with normal control flow — pip_upgrader/packages_interactive_selector.py (line 37)
  Using raise KeyboardInterrupt() to signal 'nothing to do' was a misuse of the exception — it's meant for user-initiated interrupts, not normal program flow. Returning an empty list and letting the caller handle it is the right approach.
• Guard in cli.py prevents downstream errors on empty selection — pip_upgrader/cli.py (line 76)
  The if not selected_packages: return guard cleanly prevents PackagesUpgrader from being called with an empty list and avoids a potentially confusing empty 'Updated versions' message.

Bottom line: Clean, correct fix. No issues found — approve as-is.

Automated review by reviewd in 43s. Findings are AI-generated and may not be accurate.
Replies to this comment are not monitored.

[simion]

thanks for the fix !