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
python: editorconfig fixes #96438
python: editorconfig fixes #96438
Conversation
@zowoq I noticed it a while ago CI would check for formatting. As far as I know it was never announced on the mailing list (Discourse). I think it should be, especially what the intention of this is. In the past, there was the conclusion we would not enforce style until a formatter was readily available. |
If you're referring to a Lines 7 to 12 in dbc8f5c
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Diff LGTM
looks like the rebuilds are from script blocks having their extra whitespace truncated LGTM for @FRidh 's suggestion (notification of format standardization), I think that's separate from the python modules getting cleaned up (this PR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
failures are broken on target branch
https://github.com/NixOS/nixpkgs/pull/96438
2 packages failed to build:
python37Packages.sumo python38Packages.sumo
16 packages built:
python27Packages.palettable python27Packages.rhpl python37Packages.dftfit python37Packages.lammps-cython python37Packages.nassl python37Packages.palettable python37Packages.pymatgen python37Packages.pymatgen-lammps python37Packages.sslyze python38Packages.dftfit python38Packages.lammps-cython python38Packages.nassl python38Packages.palettable python38Packages.pymatgen python38Packages.pymatgen-lammps sslyze
Motivation for this change
The
editorconfig-check
hasn't been reenabled yet (probably in the next couple of days) but I though I'd try to get this in while it isn't causing conflicts with staging/staging-next.This covers everything in
python-modules
.Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)