PR: Initial Editor migration to the new API

[dalthviz]

Description of Changes

Move Editor to use newest plugins API

Issue(s) Resolved

Fixes #12193
Part of #17468

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

[pep8speaks]

Hello @dalthviz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file runtests.py:

Line 36:80: E501 line too long (88 > 79 characters)

• In the file spyder/app/tests/conftest.py:

Line 485:80: E501 line too long (92 > 79 characters)
Line 487:80: E501 line too long (81 > 79 characters)

• In the file spyder/app/tests/test_mainwindow.py:

Line 3012:80: E501 line too long (81 > 79 characters)

• In the file spyder/plugins/editor/tests/conftest.py:

Line 43:80: E501 line too long (88 > 79 characters)
Line 132:80: E501 line too long (84 > 79 characters)
Line 133:80: E501 line too long (81 > 79 characters)
Line 136:1: W293 blank line contains whitespace

• In the file spyder/plugins/editor/tests/test_plugin.py:

Line 274:80: E501 line too long (81 > 79 characters)
Line 293:80: E501 line too long (80 > 79 characters)
Line 317:80: E501 line too long (80 > 79 characters)
Line 321:80: E501 line too long (80 > 79 characters)
Line 336:80: E501 line too long (80 > 79 characters)

• In the file spyder/plugins/editor/widgets/editorstack/editorstack.py:

Line 1750:80: E501 line too long (83 > 79 characters)

• In the file spyder/plugins/editor/widgets/main_widget.py:

Line 1887:13: E731 do not assign a lambda expression, use a def
Line 2072:9: E731 do not assign a lambda expression, use a def

• In the file spyder/plugins/editor/widgets/tests/test_editorsplitter.py:

Line 145:80: E501 line too long (80 > 79 characters)

Comment last updated at 2024-02-27 20:31:48 UTC

[ccordoba12]

Thanks @dalthviz for your hard work on this one!

[ccordoba12]

Why this shortcut is not global anymore?

[dalthviz]

Because this shortcut is defined under editor (and it has been like that from even before the changes done in this PR):

spyder/spyder/config/main.py

Line 488 in f109640

'editor/run selection and advance': "F9",

I think I did this test change most probabIy since I have been running the tests locally and I stumbled across this one failing. Also, cheking this test definition more closely, I think is not being run on the CI. Most probably the reason is that the shortcut change (from _ to editor + the shortcut string/name change) was done in the past but the test was not updated and instead skipped?. Should I also remove the CI skip for the test?

[ccordoba12]

Thanks @dalthviz for your hard work and patience to finally migrate the Editor to the new API!

Note: The failure in our tests is unrelated to this.