Skip to content

fix: stop manual spinners in link, init, and repo commands on Ctrl+C … - #124

Merged
Huerte merged 1 commit into
mainfrom
fix/manual-spinner-ctrl-c-handlers
Aug 7, 2026
Merged

fix: stop manual spinners in link, init, and repo commands on Ctrl+C …#124
Huerte merged 1 commit into
mainfrom
fix/manual-spinner-ctrl-c-handlers

Conversation

@Huerte

@Huerte Huerte commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Type

  • Bug fix
  • New feature
  • Documentation
  • Refactor / internal

Changes

  • ssh_utils.py: wrapped _get_cached_ssh_response and _get_ssh_response inside check_connection with a KeyboardInterrupt handler so the manual SSH check spinner stops cleanly if the user hits Ctrl+C during gitgo link.
  • repo.py: added a KeyboardInterrupt handler to repo_operation to ensure the repository creation spinner stops properly.
  • init.py: added KeyboardInterrupt handlers to both the download and extraction phases of _download_and_extract_template so the template fetching spinner halts cleanly on interrupt.

Checklist

  • Tested locally and changes work as expected
  • CHANGELOG.md updated under [Unreleased] (Note: Updated directly in 1.10.3 for patch republish)
  • README.md updated (if a command was added or changed)
  • Tests added or updated (if applicable)
  • No existing commands are broken (if they are, describe the impact above)

@Huerte
Huerte merged commit 1db50e5 into main Aug 7, 2026
9 checks passed
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 37.50000% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pygitgo/auth/ssh_utils.py 42.85% 4 Missing ⚠️
src/pygitgo/commands/init.py 33.33% 4 Missing ⚠️
src/pygitgo/commands/repo.py 33.33% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

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