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

Fix undefined variable in tested dict comprehensions. - #976

Merged
Mikhail Arkhipov (MikhailArkhipov) merged 40 commits into
microsoft:masterfrom
MikhailArkhipov:960
Apr 17, 2019
Merged

Mikhail Arkhipov (MikhailArkhipov) merged 40 commits into
microsoft:masterfrom
MikhailArkhipov:960

Conversation

@MikhailArkhipov

Copy link
Copy Markdown

Fixes #960

The bug was that walker returned true causing node to be walked again but not without names.

MikhailArkhipov added 30 commits March 1, 2019 09:51
This reverts commit 7ffc9db.
@MikhailArkhipov
Mikhail Arkhipov (MikhailArkhipov) merged commit 593aeef into microsoft:master Apr 17, 2019
Jake Bailey (jakebailey) pushed a commit to jakebailey/python-language-server that referenced this pull request Nov 1, 2019
* Fix microsoft#668 (partial)

* Tests

* Revert "Tests"

This reverts commit 7ffc9db.

* Exp

* Limit concurrency

* Concurrency limit

* Drop cache after analysis

* Fix regression

* Fix test

* Exclude empty names

* PR feedback

* Do not walk node twice
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.

Undefined variable in dict comprehension

2 participants