Skip to content

Fix banner unicode crashes and add git repository checks to commands - #118

Merged
Huerte merged 2 commits into
mainfrom
fix/banner-unicode-and-non-git
Jul 27, 2026
Merged

Fix banner unicode crashes and add git repository checks to commands#118
Huerte merged 2 commits into
mainfrom
fix/banner-unicode-and-non-git

Conversation

@Huerte

@Huerte Huerte commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Type

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

Changes

  • banner.py: fixed unicode crashes on older Windows terminals, made all box borders consistently green, and added a friendly message when run outside a git repo instead of throwing an error.
  • gitgo push, pull, state, undo, and resolve: added a check so running them outside a git repo prints a friendly error instead of crashing.
  • tests: added new tests in test_banner.py for the non-git repo behavior and updated test mocks in test_resolve.py.

Checklist

  • Tested locally and changes work as expected
  • CHANGELOG.md updated under [Unreleased]
  • 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)

@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 90.19608% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pygitgo/utils/banner.py 87.80% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Huerte
Huerte merged commit 1695436 into main Jul 27, 2026
9 checks passed
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