Skip to content

Commit

Permalink
Follow imports when going to definitions
Browse files Browse the repository at this point in the history
This fixes microsoft#1033 following the change of behaviour in jedi's
`goto_assignments` in the upgrade from 0.9.0 to 0.11.1. The
change is assumed to be a bug-fix since it now appears to obey
the `follow_imports` boolean for cases which it previously
would follow imports even when the value was set to `False`.

Since we actually do want to follow imports here, we set the
value to `True` to restore the old behaviour.
  • Loading branch information
PeterJCLaw committed Apr 2, 2018
1 parent 962c850 commit 70c8e1a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pythonFiles/completion.py
Original file line number Diff line number Diff line change
Expand Up @@ -570,7 +570,7 @@ def _process_request(self, request):
if lookup == 'definitions':
defs = []
try:
defs = self._get_definitionsx(script.goto_assignments(follow_imports=False), request['id'])
defs = self._get_definitionsx(script.goto_assignments(follow_imports=True), request['id'])
except:
pass
try:
Expand Down

0 comments on commit 70c8e1a

Please sign in to comment.