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: Bump pylint to >=3.1,4 and python-lsp-server to >=1.11.0,<1.12.0 #21942

Merged
merged 4 commits into from
Mar 30, 2024

Conversation

bnavigator
Copy link
Contributor

@bnavigator bnavigator commented Mar 29, 2024

Description of Changes

Bump Pylint and python-lsp-server to the new versions

Issue(s) Resolved

Fixes #21936

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: bnavigator

@bnavigator bnavigator changed the base branch from master to 5.x March 29, 2024 10:23
@bnavigator
Copy link
Contributor Author

Bumped pylsp because of python-lsp/python-lsp-server#543

@bnavigator bnavigator changed the title Bump pylint to >3.1,<4 Bump pylint to 3.1 and python-lsp-server to 1.11 Mar 29, 2024
@ccordoba12 ccordoba12 changed the title Bump pylint to 3.1 and python-lsp-server to 1.11 PR: Bump pylint to 3.1 and python-lsp-server to 1.11 Mar 29, 2024
@ccordoba12 ccordoba12 added this to the v5.5.4 milestone Mar 29, 2024
ccordoba12
ccordoba12 previously approved these changes Mar 29, 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.

@bnavigator, besides addressing my comment below, you also need to sync our subrepo for python-lsp-server (i.e. a copy we have of that project here).

For that, please follow these instructions, specifically this command:

git subrepo pull external-deps/python-lsp-server -r https://github.com/python-lsp/python-lsp-server.git -b develop -u -f

setup.py Show resolved Hide resolved
@ccordoba12 ccordoba12 dismissed their stale review March 29, 2024 20:32

This is not ready yet (my bad)

@pep8speaks
Copy link

pep8speaks commented Mar 29, 2024

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

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

Comment last updated at 2024-03-29 20:40:35 UTC

…rver.git --branch=develop --update --force external-deps/python-lsp-server

subrepo:
  subdir:   "external-deps/python-lsp-server"
  merged:   "4c0e99b41"
upstream:
  origin:   "https://github.com/python-lsp/python-lsp-server.git"
  branch:   "develop"
  commit:   "4c0e99b41"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"
@dalthviz dalthviz mentioned this pull request Mar 29, 2024
@ccordoba12 ccordoba12 changed the title PR: Bump pylint to 3.1 and python-lsp-server to 1.11 PR: Bump pylint to >=3.1,4 and python-lsp-server to >=1.11 Mar 30, 2024
@ccordoba12 ccordoba12 changed the title PR: Bump pylint to >=3.1,4 and python-lsp-server to >=1.11 PR: Bump pylint to >=3.1,4 and python-lsp-server to >=1.11.0,<1.12.0 Mar 30, 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.

Looks good to me now, thanks @bnavigator!

Note: The failure in our tests was fixed in PR #21939.

@ccordoba12 ccordoba12 merged commit 26244db into spyder-ide:5.x Mar 30, 2024
21 of 24 checks passed
ccordoba12 added a commit that referenced this pull request Mar 30, 2024
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Apr 2, 2024
https://build.opensuse.org/request/show/1163542
by user bnavigator + anag+factory
- Update to 5.5.3
  * Fix critical error when starting IPython console kernels on
    macOS and Linux in conda environments.
- Release 5.5.2
  * Activate environment for consoles that run in the same
    environment as Spyder.
  * Fix IPython kernel hangs when using a different graphic backend
    than Inline.
  * Fix check for updates mechanism when running Spyder from a
    conda installation.
- Add spyder-pr21939-qtawesome1.3.patch gh#spyder-ide/spyder#21939
  but also allow 1.2
- Add spyder-pr21942-pylint3.1.patch gh#spyder-ide/spyder#21942
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