Skip to content

handle no-tty case in gum_func with our old/better error#115

Merged
mxcl merged 3 commits intomainfrom
fix-no-tty
Jan 14, 2023
Merged

handle no-tty case in gum_func with our old/better error#115
mxcl merged 3 commits intomainfrom
fix-no-tty

Conversation

@jhheider
Copy link
Copy Markdown
Contributor

@jhheider jhheider commented Jan 6, 2023

No description provided.

@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Jan 6, 2023

  • The function gum_func() was added.
  • A new elif statement was added to the if-statement in line 214, which checks whether stdout is not a terminal (i.e., it's redirected). If so, then return 1 and exit the script with an error code of 1; otherwise continue executing as normal by returning 0 and exiting normally without any errors or warnings.

@jhheider jhheider changed the title handle no-tty case in gum_func handle no-tty case in gum_func with our old/better error Jan 6, 2023
@jhheider jhheider requested a review from mxcl January 6, 2023 23:54
@jhheider jhheider linked an issue Jan 6, 2023 that may be closed by this pull request
@mxcl mxcl merged commit 51a497d into main Jan 14, 2023
@mxcl mxcl deleted the fix-no-tty branch January 14, 2023 17:50
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.

gum_no_tty not working

2 participants