Skip to content

Fix ParameterInformation#135

Merged
karthiknadig merged 5 commits into
microsoft:mainfrom
r3m0t:parameterinformation-signature
Dec 20, 2022
Merged

Fix ParameterInformation#135
karthiknadig merged 5 commits into
microsoft:mainfrom
r3m0t:parameterinformation-signature

Conversation

@r3m0t
Copy link
Copy Markdown
Contributor

@r3m0t r3m0t commented Dec 19, 2022

Fix for jedi-language-server

@karthiknadig
Copy link
Copy Markdown
Member

@r3m0t can you run python -m nox format that should address all formatting and linting issues.

@karthiknadig karthiknadig self-assigned this Dec 19, 2022
@karthiknadig karthiknadig added the bug Issue identified by VS Code Team member as probable bug label Dec 19, 2022
@karthiknadig karthiknadig self-requested a review December 19, 2022 19:40
@r3m0t
Copy link
Copy Markdown
Contributor Author

r3m0t commented Dec 19, 2022

Done python -m nox -s format, it changed a lot of other files as well but I didn't commit those changes. Odd.

@karthiknadig karthiknadig enabled auto-merge (squash) December 19, 2022 20:58
karthiknadig
karthiknadig previously approved these changes Dec 19, 2022
@karthiknadig
Copy link
Copy Markdown
Member

Looks like docformatter has made minor changes to how it formats things. I will push an update to this PR.

@r3m0t
Copy link
Copy Markdown
Contributor Author

r3m0t commented Dec 19, 2022

Ah, Python, where three lockfiles are still not enough 🤣

@karthiknadig karthiknadig merged commit 4e2b185 into microsoft:main Dec 20, 2022
@brettcannon
Copy link
Copy Markdown
Member

Ah, Python, where three lockfiles are still not enough 🤣

I'm working on it. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue identified by VS Code Team member as probable bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants