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

PR: Add backend for a new Remote client plugin #21757

Merged
merged 45 commits into from
May 11, 2024

Conversation

hlouzada
Copy link
Contributor

@hlouzada hlouzada commented Feb 1, 2024

Description of Changes

  • Develop a new plugin that is responsible for communicating with and installing the spyder-remote-server in order to execute code in remote servers.
  • This is part of the first stage of our CZI grant.

Issue(s) Resolved

Part of #10946

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: @hlouzada

@pep8speaks
Copy link

pep8speaks commented Feb 1, 2024

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

Line 278:6: E131 continuation line unaligned for hanging indent
Line 279:6: E128 continuation line under-indented for visual indent
Line 280:6: E128 continuation line under-indented for visual indent
Line 281:6: E128 continuation line under-indented for visual indent

Line 53:80: E501 line too long (114 > 79 characters)
Line 54:80: E501 line too long (104 > 79 characters)
Line 55:80: E501 line too long (123 > 79 characters)
Line 56:80: E501 line too long (121 > 79 characters)
Line 57:80: E501 line too long (125 > 79 characters)

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

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

Line 53:80: E501 line too long (93 > 79 characters)
Line 89:61: W605 invalid escape sequence '\m'
Line 89:80: E501 line too long (82 > 79 characters)
Line 91:78: W605 invalid escape sequence '\m'
Line 91:80: E501 line too long (92 > 79 characters)
Line 93:59: W605 invalid escape sequence '\m'
Line 98:39: W605 invalid escape sequence '\m'
Line 100:69: W605 invalid escape sequence '\m'
Line 100:80: E501 line too long (146 > 79 characters)

Comment last updated at 2024-05-05 00:18:18 UTC

@hlouzada hlouzada changed the title feat: initial commit for spyder remote client [WIP] PR: Add remote client plugin Feb 1, 2024
@hlouzada hlouzada force-pushed the add_remote_client_plugin branch 2 times, most recently from adf1ddc to 2dd27fb Compare March 4, 2024 20:26
@hlouzada hlouzada force-pushed the add_remote_client_plugin branch 2 times, most recently from 2d2211b to a088501 Compare March 5, 2024 15:53
@hlouzada hlouzada force-pushed the add_remote_client_plugin branch 2 times, most recently from e770759 to 27ed1e2 Compare March 27, 2024 21:27
@ccordoba12 ccordoba12 force-pushed the add_remote_client_plugin branch 2 times, most recently from ee72159 to 7c7e757 Compare April 18, 2024 02:38
@ccordoba12 ccordoba12 added this to the v6.0beta1 milestone May 4, 2024
@ccordoba12 ccordoba12 changed the title [WIP] PR: Add remote client plugin PR: Add backend fo a new Remote client plugin May 4, 2024
@ccordoba12 ccordoba12 changed the title PR: Add backend fo a new Remote client plugin PR: Add backend for a new Remote client plugin May 5, 2024
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @hlouzada for your hard work on this!

Note: The graphical interface to make use of this backend will come in a follow-up PR.

@ccordoba12 ccordoba12 merged commit e7362e8 into spyder-ide:master May 11, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants