Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: make sure the interactive printers can cleanup after Ctrl+C #383

Merged
merged 2 commits into from Jul 26, 2022
Merged

fix: make sure the interactive printers can cleanup after Ctrl+C #383

merged 2 commits into from Jul 26, 2022

Commits on Jul 25, 2022

  1. fix: make sure the interactive printers can cleanup after Ctrl+C 

    After canceling the interactive printers with Ctrl+C, `os.Exit` was
directly called in the keyboard listener. Because of this the
`defer` statements of the surrounding function were not called
(for example `cursor.Show`) and the keyboard listener
was still running.
This resulted in a broken user "terminal" (no Cursor, no Ctrl+C)
    @jochil
    jochil committed Jul 25, 2022
    Copy the full SHA
    be55629 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2022

  1. Merge branch 'master' into jochil/cleanup-after-cancelation

    @MarvinJWendt
    MarvinJWendt committed Jul 26, 2022
    Copy the full SHA
    63b4ba3 View commit details
    Browse the repository at this point in the history