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

Loading of old third-party plugins in Spyder 6 #20776

Closed
jitseniesen opened this issue Apr 5, 2023 · 1 comment · Fixed by #20789
Closed

Loading of old third-party plugins in Spyder 6 #20776

jitseniesen opened this issue Apr 5, 2023 · 1 comment · Fixed by #20789
Assignees
Milestone

Comments

@jitseniesen
Copy link
Member

Do we still need the code in

self.set_splash(_("Loading old third-party plugins..."))
and further?

This code tries to load plugins like spyder-unittest, even if they are already loaded in the block just above ("Instantiate external Spyder 5 plugins"). The attempt to load spyder-unittest as an old third-party plugin silently fails and it does not seem to affect Spyder. However, there is some black magic going on in spyder/otherplugins.py which leaves a mock object in sys.modules; that smells iffy. This seems to somehow interfere with the tests that I am adding in spyder-ide/spyder-unittest#197, so I had to monkey patch Spyder to remove the loading of old third-party plugins in Spyder.

I don't know of any working "old third-party plugins" (I think this refers to plugins from Spyder 4), so how about removing that code? I think that this is the only place that uses spyder/otherplugins.py so we can remove that file too.

@jitseniesen jitseniesen added this to the v6.0.1 milestone Apr 5, 2023
@ccordoba12 ccordoba12 modified the milestones: v6.0.1, v6.0alpha1 Apr 5, 2023
@ccordoba12 ccordoba12 self-assigned this Apr 5, 2023
@ccordoba12
Copy link
Member

I don't know of any working "old third-party plugins" (I think this refers to plugins from Spyder 4)

Right, I think so too.

so how about removing that code? I think that this is the only place that uses spyder/otherplugins.py so we can remove that file too.

Good idea. Will do right away.

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