Remove calls to process.exit() which forces async calls to exit early #15

kwkelly · 2018-06-20T18:30:38Z

Fixes #10

fixes bitwarden#7

kwkelly · 2018-06-20T18:39:47Z

@kspearrin I didn't have the latest version of the jslib submodule on this PR. We might want to revert this briefly.

kspearrin · 2018-06-20T18:40:34Z

Ah, didn't notice the jslib changes. Will revert.

…it early (#15)" This reverts commit 87be2e8.

kspearrin · 2018-06-20T18:42:52Z

@kwkelly I re-did it here: 6fbcfca

Though it looks like mine did a jslib update as well.

kwkelly added 5 commits June 17, 2018 17:23

Update invocations of inquirer.js prompts to output prompt to stderr.

e6a8430

fixes bitwarden#7

Merge branch 'master' of https://github.com/bitwarden/cli

ad5bfc4

Updates to use updated typings from @types/inquirer

9c31f93

Merging conflicts and removing calls to 'process.exit()'

834122d

Removes extraneous async / await from writeLn

243cc66

kspearrin merged commit 87be2e8 into bitwarden:master Jun 20, 2018

kspearrin added a commit that referenced this pull request Jun 20, 2018

Revert "Remove calls to process.exit() which forces async calls to ex…

2418e20

…it early (#15)" This reverts commit 87be2e8.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove calls to process.exit() which forces async calls to exit early #15

Remove calls to process.exit() which forces async calls to exit early #15

kwkelly commented Jun 20, 2018

kwkelly commented Jun 20, 2018

kspearrin commented Jun 20, 2018

kspearrin commented Jun 20, 2018 •

edited

Loading

Remove calls to process.exit() which forces async calls to exit early #15

Remove calls to process.exit() which forces async calls to exit early #15

Conversation

kwkelly commented Jun 20, 2018

kwkelly commented Jun 20, 2018

kspearrin commented Jun 20, 2018

kspearrin commented Jun 20, 2018 • edited Loading

kspearrin commented Jun 20, 2018 •

edited

Loading