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.

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

error message printed to stderr, exit with error code on failure #14

Merged
merged 1 commit into from
Feb 17, 2015

Conversation

WnP
Copy link
Contributor

@WnP WnP commented Feb 11, 2015

Features:

  • error messages are now printed to stderr
  • when program failed it return an error code

error code use case:

stackit -e "python broken_app.py" && vim broken_app.py

if no error question is return by stackit vim is not launched

if questions are return I can found a solution on StackOverflow and fix my broken_app.py using vim

if not questions:
click.echo(
click.style("Your search \'{0}\' with tags \'{1}\' returned no results.".format(config.term,config.tag),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

PEP8: missing whitespace after ','

@lukasschwab
Copy link
Owner

@WnP I like this, and it should hopefully be an easy pull request to evaluate.

Unfortunately, I'm super swamped right now, hence the delay––I'll try to review this by EOD, but it might take until tomorrow.

@lukasschwab
Copy link
Owner

Works beautifully––merging!

lukasschwab added a commit that referenced this pull request Feb 17, 2015
error message printed to stderr, exit with error code on failure
@lukasschwab lukasschwab merged commit 8ab6c4b into lukasschwab:master Feb 17, 2015
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.

2 participants