Skip to content

Commit

Permalink
chore(tui): remove unwraps from tui (#9034)
Browse files Browse the repository at this point in the history
### Description

As #9016 displayed, panicking
in the render thread leads to a very poor user experience. We should do
our best effort to avoid this by using errors so we can make sure the
`cleanup` code gets run.

This PR removes (almost) all `unwrap`s/`expect`s in favor of returning
an `Err` which will cause us to exit `run_app_inner`.

### Testing Instructions

Existing unit tests. 👀
  • Loading branch information
chris-olszewski authored Aug 20, 2024
1 parent 452a916 commit d9bd4f0
Show file tree
Hide file tree
Showing 3 changed files with 164 additions and 126 deletions.
Loading

0 comments on commit d9bd4f0

Please sign in to comment.