Skip to content

refactor(terminal): install the panic hook from inside the guard - #10

Merged
greenbugx merged 1 commit into
mainfrom
refactor/panic-hook-ownership
Aug 26, 2026
Merged

greenbugx merged 1 commit into
mainfrom
refactor/panic-hook-ownership

Conversation

@greenbugx

Copy link
Copy Markdown
Owner

Summary

Related Issue

Closes #

Type of Change

  • feat — new feature
  • fix — bug fix
  • docs — documentation only
  • refactor — no change in behavior
  • test — tests only
  • chore / ci — tooling, dependencies, workflows
  • Breaking change — an existing keybinding, CLI flag, or output changes meaning

How This Was Tested

  • Automated:
  • By hand:
  • Tested on: OS [e.g. Windows 11 / Ubuntu 24.04 / macOS 15], terminal [e.g. Windows Terminal / Alacritty / Kitty]

Screenshots / Recordings

Checklist

  • cargo fmt — CI runs cargo fmt --check
  • cargo clippy --all-targets -- -D warnings is clean, unused imports included
  • cargo test passes
  • Commits follow Conventional Commits
  • Files are LF, not CRLF
  • Nothing used that is newer than the rust-version floor in Cargo.toml
  • A new module is listed in CONTRIBUTING.md's architecture overview
  • New keybinding added to ui::help::SECTIONS, to the key list in its test, and to the README table

Notes for the Reviewer

Repository owner deleted a comment from chatgpt-codex-connector Bot Aug 26, 2026
@greenbugx
greenbugx merged commit 19cf36a into main Aug 26, 2026
4 checks passed
@greenbugx
greenbugx deleted the refactor/panic-hook-ownership branch August 26, 2026 18:08
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.

1 participant