Skip to content
This repository has been archived by the owner on Nov 11, 2023. It is now read-only.

CLI improvements #155

Merged
merged 3 commits into from
Oct 15, 2019
Merged

CLI improvements #155

merged 3 commits into from
Oct 15, 2019

Conversation

TejasQ
Copy link
Contributor

@TejasQ TejasQ commented Oct 15, 2019

Why

This PR fixes #154 and also makes the --output option optional, printing to standard output in case there is no output specified. This is useful for dry runs and a common pattern for CLI tools (see babel).

@contiamo contiamo deleted a comment from contiamo-ci Oct 15, 2019
Copy link
Contributor

@stereobooster stereobooster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks ok

@TejasQ TejasQ merged commit db0636f into master Oct 15, 2019
@TejasQ TejasQ deleted the cli-improvements branch October 15, 2019 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect exit code on errors
2 participants