Skip to content

Print an error message when doctr fails  #239

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

Merged
asmeurer merged 10 commits into from
Aug 22, 2017
Merged

Print an error message when doctr fails  #239

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
037c0a5
Print an error message when doctr fails
asmeurer Aug 22, 2017
1d995db
Add an exit early test, to make test out the error message code
asmeurer Aug 22, 2017
f79d0ae
Print the error message in red
asmeurer Aug 22, 2017
4649794
Print the exception message in red
asmeurer Aug 22, 2017
5a5ae45
Put the error message on the same line
asmeurer Aug 22, 2017
241e0bc
Merge branch 'master' into error-message
asmeurer Aug 22, 2017
4824ad3
Fix syntax
asmeurer Aug 22, 2017
e0c03ee
Merge branch 'error-message' of github.com:drdoctr/doctr into error-m…
asmeurer Aug 22, 2017
443aa3e
Revert "Add an exit early test, to make test out the error message code"
asmeurer Aug 22, 2017
3662a3e
Merge branch 'master' into error-message
asmeurer Aug 22, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions doctr/__main__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@
import subprocess
import yaml
import json
import shlex

from pathlib import Path

Expand All @@ -40,6 +41,9 @@
get_travis_branch, copy_to_tmp, checkout_deploy_branch)
from . import __version__

def red(text):
return "\033[31m%s\033[0m" % text

def make_parser_with_config_adder(parser, config):
"""factory function for a smarter parser:

Expand Down Expand Up @@ -290,6 +294,11 @@ def deploy(args, parser):
print("Don't have permission to push. Not trying.")
else:
print("The docs have not changed. Not updating")
except BaseException as e:
DOCTR_COMMAND = ' '.join(map(shlex.quote, sys.argv))
print(red("ERROR: The doctr command %r failed: %s" % (DOCTR_COMMAND, e)),
file=sys.stderr)
raise
finally:
run(['git', 'checkout', current_commit])
# Ignore error, won't do anything if there was nothing to stash
Expand Down