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

Missing symbols in autoimport.db #687

Closed
lieryan opened this issue Apr 18, 2023 Discussed in #685 · 0 comments · Fixed by #688
Closed

Missing symbols in autoimport.db #687

lieryan opened this issue Apr 18, 2023 Discussed in #685 · 0 comments · Fixed by #688
Labels
autoimport bug Unexpected or incorrect user-visible behavior

Comments

@lieryan
Copy link
Member

lieryan commented Apr 18, 2023

Discussed in #685

Originally posted by rstcruzo April 6, 2023
Hey everyone. I have been using rope with pylsp for a couple of days and noticed something odd. Autoimport suggestions contain my virtual env packages if I add the virtual env site-packages folder to the python_path preference. That's okay, but seems like rope does not scan all subfolders to generate autoimports.

For instance, I'm using prompt_toolkit, and the prompt_toolkit.prompt is returned by rope to auto import it, but the prompt_toolkit.completion.Completer is not. prompt_toolkit.history.FileHistory is listed too. So the behavior is inconsistent.

Seems to me that rope is not able to autoimport from folders inside the prompt_toolkit, since completion is a folder, history is a python file. Is this expected? or is there any configuration I can set to fix this?

I also checked the autoimport.db file and looks like the folders of prompt_toolkit are missing from there too.

@lieryan lieryan added bug Unexpected or incorrect user-visible behavior autoimport labels Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autoimport bug Unexpected or incorrect user-visible behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant