spyder icon indicating copy to clipboard operation
spyder copied to clipboard

PR: Unify kernel configuration

Open impact27 opened this issue 2 years ago • 1 comments

Description of Changes

  • [ ] Wrote at least one-line docstrings (for any new functions)
  • [ ] Added unit test(s) covering the changes (if testable)
  • [ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Unify two ways of changing the kernel configuration. This also avoids restarting the cached kernel if the options have been changed.

Issue(s) Resolved

Fixes #

Spyder-kernels: https://github.com/spyder-ide/spyder-kernels/pull/483

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:

impact27 avatar Mar 17 '24 06:03 impact27

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

Line 28:1: E302 expected 2 blank lines, found 1 Line 33:28: W292 no newline at end of file

Comment last updated at 2024-03-17 08:10:36 UTC

pep8speaks avatar Mar 17 '24 06:03 pep8speaks