-
Notifications
You must be signed in to change notification settings - Fork 50
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
Question: does it raise for invalid docstrings? #35
Comments
Looks like this issue is hitting timothycrosley/pdocs#19 as well, ideally a misparse should be caught and raised as a |
Sorry for the late reply. It's supposed to raise a ParseError, if it does not, it's an error and we need specific input to debug it. |
Here is a poorly formatted doc string that would hit this: def test(param: str) -> str:
"""This is a test
Args:
param - poorly formatted
"""
return param |
Thanks @Peter200lx , I can confirm that 0.8 still had a problem with this kind of input and the package now raises a ParseError as of 0.8.1. |
Hi,
Thanks for your library it looks awesome. Does the parser raise an exception if the docstring's syntax is not correct Google-style syntax?
(Trying to build a simple docstring CI validation in my repo)
Thanks!
The text was updated successfully, but these errors were encountered: