Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Fix import * over local declarations#1433

Merged
jakebailey merged 4 commits intomicrosoft:masterfrom
jakebailey:import-overtop
Aug 13, 2019
Merged

Fix import * over local declarations#1433
jakebailey merged 4 commits intomicrosoft:masterfrom
jakebailey:import-overtop

Conversation

@jakebailey
Copy link
Member

Fixes #1257.

Cherry picked out of #1429.

@jakebailey jakebailey changed the title Fix import * over local declarations [WIP] Fix import * over local declarations Aug 13, 2019
@jakebailey
Copy link
Member Author

WIP until FromImportMultiValues test is fixed.

@jakebailey jakebailey changed the title [WIP] Fix import * over local declarations Fix import * over local declarations Aug 13, 2019
@jakebailey
Copy link
Member Author

Test suite passes with that extra commit.

@jakebailey jakebailey merged commit 1241b6e into microsoft:master Aug 13, 2019
@jakebailey jakebailey deleted the import-overtop branch August 13, 2019 23:16
jakebailey added a commit to jakebailey/python-language-server that referenced this pull request Nov 1, 2019
* Fix import * over local declarations (Socket)

* Add a test

* Add handling of import position relative to the variable


Co-authored-by: Mikhail Arkhipov <mikhaila@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

* imported functions are not overwritten by functions of the same name

2 participants