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

Add docstring_parser 0.16 #122

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

wusteven815
Copy link
Contributor

@wusteven815 wusteven815 commented Oct 4, 2024

@@ -19,6 +19,7 @@ java-utilities

# optional autocomplete feature
jedi==0.19.1
docstring_parser==0.16
Copy link
Member

Choose a reason for hiding this comment

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

We typically prefer to be more lenient with our "requirements"; jedi is a special case where we have seen bad things happen with different versions, it's the only version we pin in py/server/setup.py. Do you have reason to believe that earlier versions of docstring_parer won't work? Or, that future versions of docstring_parser will be incompatible? If not, it would be preferable to not set a version (or, if you know old versions don't work, set a minimum version).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated it to be >=0.16. I'm using a feature that's new in this version and there's no reason to believe that future versions would be incompatible.

@devinrsmith devinrsmith merged commit 7478782 into deephaven:main Oct 4, 2024
3 checks passed
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