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

PR: Put user provided paths at the front of sys.path #139

Closed
wants to merge 1 commit into from

Conversation

mrclary
Copy link
Contributor

@mrclary mrclary commented Dec 22, 2021

Put user provided paths (via extra_paths) at the front of sys.path instead of the back.

Similarly for the document path, if requested.
Completions search path is then: Document.path + extra_paths + Document._extra_sys_path + environment sys.path

See spyder-ide/spyder#17066

Copy link
Contributor

@krassowski krassowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks good to me, but I will refrain from merging due to an ongoing discussion in spyder-ide/spyder#17066

@mrclary mrclary marked this pull request as draft December 30, 2021 17:46
…stead of the back.

Similarly for the document path, if requested.
Completions search path is then: Document.path + extra_paths + Document._extra_sys_path + environment sys.path
@simson
Copy link

simson commented Sep 9, 2022

Hi,

I'm facing the same issue. I don't understand the link with spyder-ide and why this PR is now closed.

I use python-lsp in neovim and I would be a convenient solution for project including sub-packages.

@mrclary
Copy link
Contributor Author

mrclary commented Sep 14, 2022

Hi,

I'm facing the same issue. I don't understand the link with spyder-ide and why this PR is now closed.

I use python-lsp in neovim and I would be a convenient solution for project including sub-packages.

@simson, there may be an option to do this in a future release of Spyder, when we migrate the PYTHONPATH Manager to its own plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants