Describe the bug
Black formatting issue.
https://hydra.nixos.org/build/130880394
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
Hmm, if a package bump of black can have this kind of impact, I'm wondering if we should reconsider having a mandatory style check in our VM tests.
We want to have a common style, so we use a tool. If a tool like Black has this type of "bug", there is no way around it, and it becomes your new style. If the "bug" is then resolved, you need to update your code.
I think this is annoying, but we should not make a too big of a deal out of it. Unless this of course becomes a regular thing.
I think this is annoying, but we should not make a too big of a deal out of it. Unless this of course becomes a regular thing.
Would not be much of an issue if black was a good fit in the first place — because of variable expansion, there is no good way to reformat a NixOS VM test defined as a Nix string with some variables. This is bad even without upstream bugs.
Most helpful comment
Would not be much of an issue if black was a good fit in the first place — because of variable expansion, there is no good way to reformat a NixOS VM test defined as a Nix string with some variables. This is bad even without upstream bugs.