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

python310Packages.autopep8: 1.7.1 -> 2.0.0 #198486

Merged

Conversation

r-ryantm
Copy link
Contributor

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.

meta.description for python310Packages.autopep8 is: "A tool that automatically formats Python code to conform to the PEP 8 style guide"

meta.homepage for python310Packages.autopep8 is: "https://pypi.org/project/autopep8/"

meta.changelog for python310Packages.autopep8 is: ""

Updates performed
  • Ran passthru.UpdateScript
To inspect upstream changes
Impact
Checks done (click to expand)
  • built on NixOS
  • The tests defined in passthru.tests, if any, passed
  • found 2.0.0 with grep in /nix/store/pakzgf829jzwzkww6c2y2jm68k522z1d-python3.10-autopep8-2.0.0
  • found 2.0.0 in filename of file in /nix/store/pakzgf829jzwzkww6c2y2jm68k522z1d-python3.10-autopep8-2.0.0

Rebuild report (if merged into master) (click to expand)
21 total rebuild path(s)

21 package rebuild(s)

First fifty rebuilds by attrpath
cmake-format
cmake-language-server
python310Packages.autopep8
python310Packages.django_silk
python310Packages.pyls-flake8
python310Packages.pyls-isort
python310Packages.pyls-spyder
python310Packages.pylsp-mypy
python310Packages.python-lsp-black
python310Packages.python-lsp-server
python310Packages.spyder
python39Packages.autopep8
python39Packages.django_silk
python39Packages.pyls-flake8
python39Packages.pyls-isort
python39Packages.pyls-spyder
python39Packages.pylsp-mypy
python39Packages.python-lsp-black
python39Packages.python-lsp-server
python39Packages.spyder
spyder
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/pakzgf829jzwzkww6c2y2jm68k522z1d-python3.10-autopep8-2.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A python310Packages.autopep8 https://github.com/r-ryantm/nixpkgs/archive/de1d155444abea8635b7f74999e7ca24488e176a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/pakzgf829jzwzkww6c2y2jm68k522z1d-python3.10-autopep8-2.0.0
ls -la /nix/store/pakzgf829jzwzkww6c2y2jm68k522z1d-python3.10-autopep8-2.0.0/bin


Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:
  • spyder (python310Packages.spyder)
  • python39Packages.spyder
18 packages built:
  • cmake-format
  • cmake-language-server
  • python310Packages.autopep8
  • python310Packages.django_silk
  • python310Packages.pyls-flake8
  • python310Packages.pyls-isort
  • python310Packages.pyls-spyder
  • python310Packages.pylsp-mypy
  • python310Packages.python-lsp-black
  • python310Packages.python-lsp-server
  • python39Packages.autopep8
  • python39Packages.django_silk
  • python39Packages.pyls-flake8
  • python39Packages.pyls-isort
  • python39Packages.pyls-spyder
  • python39Packages.pylsp-mypy
  • python39Packages.python-lsp-black
  • python39Packages.python-lsp-server

Maintainer pings

cc @bjornfor for testing.

@SFrijters
Copy link
Member

The toml dependency was replaced by tomli (already in 1.7.1), this was causing me some headache earlier.
Would be nice to fix this in this update: https://github.com/hhatto/autopep8/releases/tag/v1.7.1

@SFrijters SFrijters self-requested a review November 10, 2022 19:30
@bjornfor
Copy link
Contributor

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

20 packages built:
  • cmake-format
  • cmake-language-server
  • python310Packages.autopep8
  • python310Packages.django_silk
  • python310Packages.pyls-flake8
  • python310Packages.pyls-isort
  • python310Packages.pyls-spyder
  • python310Packages.pylsp-mypy
  • python310Packages.python-lsp-black
  • python310Packages.python-lsp-server
  • spyder (python310Packages.spyder)
  • python39Packages.autopep8
  • python39Packages.django_silk
  • python39Packages.pyls-flake8
  • python39Packages.pyls-isort
  • python39Packages.pyls-spyder
  • python39Packages.pylsp-mypy
  • python39Packages.python-lsp-black
  • python39Packages.python-lsp-server
  • python39Packages.spyder

@bjornfor
Copy link
Contributor

The toml dependency was replaced by tomli (already in 1.7.1), this was causing me some headache earlier. Would be nice to fix this in this update: https://github.com/hhatto/autopep8/releases/tag/v1.7.1

* Use tomli in place of unmaintained toml package by @mgorny in [Use tomli in place of unmaintained toml package hhatto/autopep8#629](https://github.com/hhatto/autopep8/pull/629)

See #200578

@SFrijters SFrijters self-requested a review November 23, 2022 09:18
@bjornfor bjornfor merged commit cb3339b into NixOS:master Nov 23, 2022
@bjornfor
Copy link
Contributor

Version 1.7.1 is yanked (ref), so I think we should do a backport.

@github-actions
Copy link
Contributor

Successfully created backport PR #202502 for release-22.11.

@github-actions
Copy link
Contributor

Git push to origin failed for release-22.11 with exitcode 1

@r-ryantm r-ryantm deleted the auto-update/python310Packages.autopep8 branch November 23, 2022 12:05
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.

None yet

3 participants