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

Intellisense parameter info is incorrectly formatted if declaration is too long #4127

Closed
jeffreykxiao opened this issue Jan 24, 2019 · 4 comments
Labels
area-intellisense LSP-related functionality: auto-complete, docstrings, navigation, refactoring, etc. bug Issue identified by VS Code Team member as probable bug

Comments

@jeffreykxiao
Copy link

Environment data

  • VS Code version: 1.30.2
  • Extension version (available under the Extensions sidebar): 2018.12.1
  • OS and version: Windows 10 Pro
  • Python version (& distribution if applicable, e.g. Anaconda): 3.6.8 64-bit
  • Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
  • Relevant/affected Python packages and their versions: N/A

Expected behaviour

good
Parameter info is correctly displayed

Actual behaviour

bad
Too many parameters: incorrect display. Trailing params are parsed into docstring section. Putting extra parameters on newlines such that each line is under 70 characters doesn't help.

sad
Function declaration is too long: incorrect display.

@ghost ghost added the triage-needed Needs assignment to the proper sub-team label Jan 24, 2019
@DonJayamanne DonJayamanne removed triage-needed Needs assignment to the proper sub-team labels Jan 24, 2019
@karrtikr karrtikr added the info-needed Issue requires more information from poster label Feb 14, 2019
@karrtikr
Copy link

karrtikr commented Feb 14, 2019

Can you please confirm if you are using language server, or jedi? (Check value of python.jediEnabled in settings.json).
Also, could you provide the code?

@jeffreykxiao
Copy link
Author

Can't provide the code unfortunately, but disabling jedi solves the problem.

@karrtikr karrtikr added bug Issue identified by VS Code Team member as probable bug area-intellisense LSP-related functionality: auto-complete, docstrings, navigation, refactoring, etc. upstream-language server and removed info-needed Issue requires more information from poster triage labels Feb 15, 2019
@karrtikr karrtikr removed their assignment Feb 15, 2019
@danalf67
Copy link

danalf67 commented May 7, 2019

I have exactly the same issue in the parameter hints. I use the jedi setting because the language server is worse (all line breaks are removed after the signature). Example source code for getting the error:
import shutil
shutil.copytree()

@karrtikr
Copy link

Dup of #5577

@ghost ghost removed the needs upstream fix label May 15, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-intellisense LSP-related functionality: auto-complete, docstrings, navigation, refactoring, etc. bug Issue identified by VS Code Team member as probable bug
Projects
None yet
Development

No branches or pull requests

4 participants