-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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: Migrate the IPython Console to the new API #16324
PR: Migrate the IPython Console to the new API #16324
Conversation
Hello @dalthviz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2021-11-03 18:11:07 UTC |
d1380f8
to
bd6b397
Compare
dfc81bb
to
bcbc3e4
Compare
b7906b6
to
ab9d2d7
Compare
42f0062
to
a88911e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot @dalthviz for your hard work on this one! It looks pretty good.
This is an initial review for you, up to ipythonconsole/plugin.py
, so you can start working on it. I'll try to leave another review for what's missing before the end of the week.
032de7a
to
c196396
Compare
c15b7f5
to
a4677e1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @dalthviz, this is my second review. Things look pretty good, except for a couple of things we should take a closer look at.
ff432de
to
0bbbf9f
Compare
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
…rnel and register handlers from the Editor
…ange register/unregister kernel handlers method place to kernels section
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
…enu, MENU_SEPARATOR. Improve docstring for signals and remove commented code
…t. Add logic for conf change with sig_prompt_ready. Simplify set_elapsed_time conf change
…se update_actions instead of custom method. Remove unused code and comments
…focus_changed directly
…hen creating kernels
92a2df3
to
2550a11
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot @dalthviz for all your hard work on this! It's finally ready!
Description of Changes
IPython Console to Spyder 5 plugin API
Supersedes #12749 #15934
Depends on #15922
Issue(s) Resolved
Fixes #12194.
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