minor: rewrite/expand show_documentation #980
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
davidhalter
reviewed
Jan 10, 2020
Codecov Report
@@ Coverage Diff @@
## master #980 +/- ##
==========================================
+ Coverage 47.68% 47.83% +0.14%
==========================================
Files 3 3
Lines 799 807 +8
Branches 164 165 +1
==========================================
+ Hits 381 386 +5
- Misses 384 386 +2
- Partials 34 35 +1
Continue to review full report at Codecov.
|
blueyed
changed the title
blueyed
commented
Jan 17, 2020
| Expect header[0] == "Docstring for builtins:class ImportError" | ||
| Expect header[1] == "========================================" | ||
| endif | ||
| let content = join(getline(3, '$'), "\n") | ||
| Expect stridx(content, "Import can't find module") > 0 |
There was a problem hiding this comment.
@davidhalter with Jedi master "content" is "ImportError(*args, name: Optional[str]=..., path: Optional[str]=...)" here, i.e. the signature itself?!
git-bisected to davidhalter/jedi@4161bfc
There was a problem hiding this comment.
@blueyed For whatever reason, the documentation of some builtins doesn't seem to work anymore and the tests don't catch it. No idea why, I will investigate and fix it. I have noticed it a few days ago when documentation for open didn't show up. So really a Jedi bug.
There was a problem hiding this comment.
It works with the submodule's version, doesn't it?
There was a problem hiding this comment.
I.e. the test fails locally when updating Jedi to the above commit.
Going to merge this for now.
yangle
pushed a commit
to yangle/jedi-vim
that referenced
this pull request
Mar 8, 2020
* pythonx/jedi 60415033...35e5cf2c (106):
> A small Changelog improvement
> Cleanup Changelog for the next release
> Make it possible to access functions that were inherited, see #1347
> Messed up a Windows test
> Again a small windows issue fixed.
> Need to escape the path backslash for windows slashes
> Make it possible to access properties again
> A small compatibility fix
> Fix an issue with type vars that might have been a problem for other things as well
> Don't add quotes after paths if they are already there
> A small speed optimization that helps a lot with sys.version_info >= (3, 0) patterns in typeshed
> Remove an unnecessary isinstance usage
> Make CompiledValue lazy
> A python 2 fix
> Fix an error that occured because of some refactorings
> Fix one more os.path.join issue
> Fix more issues with os.path path completion
> Make join detection much easier
> Fix some remaining issues with file path completions
> A few more tests for path completions (join)
> Fix quote completions for os.path.join path completions
> Fix first param argument of os.path.join file completions
> Fix arglist/trailer issues
> in os.path.join completions, directories should not end in a slash
> Make some file path completions in os.path.join work
> Fix os.path.join static value gathering
> os.path.sep should always have a clear value
> Fix string name completion for stuff like dirname and abspath
> Fix bytes issue with file path adding
> Python 2 compatibility
> Make __file__ return the correct value
> Fix string additions when used in certain ways
> Make string additions work for file path completion
> Fix interpeter project path
> Fix the REPL completer for file path completions
> Fix a few more file name completion cases
> A first iteration for file path completions
> Fix a test failure
> Add Definition.execute, fixes davidhalter#1076
> Add tests to show that davidhalter#516 is not working, yet
> List possible Definition.type in its docstring, fixes davidhalter#1069.
> Start writing the Changelog for 0.15.0
> 3.8-dev should not be allowed to fail
> Skip more Python 2 tests
> Skipping of tests was done the wrong way again
> Skip the right tests
> Parameter.kind is not avaialble in Python 3.5
> Fix Python 2 test issues
> Add a few docstrings to make some things clearer
> Add ParamDefinition.kind, fixes #1361
> Make it possible to format a param to a string, fixes davidhalter#1074
> Add another test for signature annotations
> Add Signature.to_string() with proper tests, fixes davidhalter#779, fixes davidhalter#780
> Test infer_default
> Use a helper to create definitions
> Add a execute_annotation option to infer_annotation
> Test infer_annotation
> Make it possible for users to infer annotations/defaults
> A first iteration of adding signatures to the API, fixes #1139
> Fix a caching issue
> Forgot the right resolve_stars parameters in one place
> Cache getting resolved param names
> Turn around resolve_stars, it shouldn't by default be resolved
> Remove a class that is not needed anymore
> Revert "Use __str__ instead of to_string"
> Fix the last remaining issues with function signature
> overloaded_functions should be private
> Use __str__ instead of to_string
> Better repr for CallSignature
> Fix some minor signature issues
> Remove the unused function signature_matches
> Fix signatures for __init__ calls when used with supers, fixes #1163
> Fix some issues for args resolving in method calls
> Remove a bit of code that i sprobably unused
> Move get_signatures from Function to FunctionMixin
> Merge the signature changes
> Remove an import
> Small refactoring
> Skipped Python 2 Interpreter tests the wrong way
> Remove a paragraph in docs that was arguing that stubs and generics (and other things) were not properly supported, fixes davidhalter#1012
> Fix partial signatures for MixedObject
> Fix partial signatures, fixes #1371
> Make sure with a test that the staticmethod signature is also correct
> Fix classmethod signature, fixes davidhalter#498
> Deal with annotation on *args and **kwargs correctly, fixes davidhalter#980
> Use LazyContextWrapper more
> Very small refactoring
> Fix for failing assertion on native modules Issue #1354 (#1370)
> Skip numpydoc tests for Python 2
> Remove code that didn't mean anything
> Fix docstrings for wrapped functions, fixes davidhalter#906
> Fix an isue about dict ordering in Python before 3.6.
> Make it clearer when get_param is used.
> A small dataclass refactoring
> Evaluate annotations for dataclasses when infer is called on param
> Get inheritance of dataclass right
> A first working iteration of dataclass signatures, fixes #1213
> Remove an unused function
> Dataclass progress
> Add enum changes to changelog
> Fix a recursion error about getting metaclasses
> One test about recursion issues only applied to Python 2
> Use class filters in instances differently so metaclass plugins work, fixes davidhalter#1090
> First step in working with metaclasses in plugins, see davidhalter#1090.
> Refactor the plugin registry
> Remove classes from plugins and use decorators instead
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The documentation test fails with newer Jedi, this is for investigating it.
TODO:
failure on py27