You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
Do we still need the code in
spyder/spyder/app/mainwindow.py
Line 778 in 401ee99
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 loadspyder-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 insys.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.The text was updated successfully, but these errors were encountered: