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

python3Packages.python-lsp-server: fix build #183862

Merged
merged 2 commits into from
Aug 3, 2022
Merged

python3Packages.python-lsp-server: fix build #183862

merged 2 commits into from
Aug 3, 2022

Conversation

jyooru
Copy link
Member

@jyooru jyooru commented Jul 30, 2022

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@bobby285271 bobby285271 added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Jul 30, 2022
Copy link
Member

@lilyinstarlight lilyinstarlight left a comment

Choose a reason for hiding this comment

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

The postPatch section can be removed now, since those lines were moved to pyproject.toml in 1.5.0 (see python-lsp/python-lsp-server#207) but you added pytest-cov anyway which removes the need for those lines afaik

If the --junitxml=pytest.xml and mccabe lines are still relevant, though, setup.cfg should be changed to pyproject.toml on line 55

@jyooru jyooru requested a review from fabaff July 31, 2022 05:17
@bobby285271 bobby285271 removed the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Jul 31, 2022
@kira-bruneau
Copy link
Contributor

Result of nixpkgs-review pr 183862 run on x86_64-linux 1

2 packages failed to build:
  • spyder (python310Packages.spyder)
  • python39Packages.spyder
14 packages built:
  • python310Packages.pyls-flake8
  • python310Packages.pyls-isort
  • python310Packages.pyls-spyder
  • python310Packages.pylsp-mypy
  • python310Packages.python-lsp-black
  • python310Packages.python-lsp-server
  • python310Packages.whatthepatch
  • python39Packages.pyls-flake8
  • python39Packages.pyls-isort
  • python39Packages.pyls-spyder
  • python39Packages.pylsp-mypy
  • python39Packages.python-lsp-black
  • python39Packages.python-lsp-server
  • python39Packages.whatthepatch

Copy link
Contributor

@kira-bruneau kira-bruneau left a comment

Choose a reason for hiding this comment

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

spyder was already failing due to a bad version requirement, this fix LGTM! ❤️

@kira-bruneau kira-bruneau merged commit 4ef10c9 into NixOS:master Aug 3, 2022
@jyooru jyooru deleted the fix/python3Packages.python-lsp-server branch January 15, 2023 03:52
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.

6 participants