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

Raise an error with a useful message if the app is not set up. #31

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

jwg4
Copy link
Contributor

@jwg4 jwg4 commented May 24, 2017

If the app is not initialized in the Autodoc object, either through __init__ or init_app, calls to doc() seem to work correctly, there is a fairly cryptic error when calling html().

Explicitly raise an exception if self.app is not defined here, to make it easier to debug this problem.

@jwg4
Copy link
Contributor Author

jwg4 commented Sep 8, 2017

Hi, it would be great if @acoomans or someone else could take a look at this. Thanks!

@jwg4 jwg4 force-pushed the error_if_no_app branch from 18423b4 to 2bdff9c Compare May 16, 2018 14:17
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.

1 participant