nix/4.flake-parts
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Recommend touchup if formatter hack fails

Browse source
This commit is contained in:
Robert Hensing 2025-05-27 14:45:51 +02:00
parent 9d0762c18a
commit 0abe17c05d
2 changed files with 11 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

2
dev/tests/nix-unit.nix
View file

@ -583,7 +583,7 @@ in
in in
result.formatter.aarch64-darwin; result.formatter.aarch64-darwin;
expectedError.type = "ThrownError"; expectedError.type = "ThrownError";
expectedError.msg = "could not determine statically(.|\n)*disabledModules(.|\n)*inputs\\.flake-parts\\.modules\\.formatter"; expectedError.msg = "could not determine statically(.|\n)*touchup\\.attr\\.formatter\\.enable = false";
}; };
"test: empty when never defined" = { "test: empty when never defined" = {

19
modules/formatter.nix
View file

@ -85,16 +85,17 @@ in
This whole situation should be temporary. `nix flake check`/`show` can be changed to allow `null` values, which gives flake-parts and other frameworks a way to avoid this situation. This whole situation should be temporary. `nix flake check`/`show` can be changed to allow `null` values, which gives flake-parts and other frameworks a way to avoid this situation.
If you don't want a `formatter` output attribute, you may remove it in your `mkFlake` call. To change the `formatter` output attribute, you can control it precisely with the `touchup` module, for example:
(`disabledModules` must not be used in modules that are intended for reuse)
{ inputs, ... }: imports = [ inputs.flake-parts.flakeModules.touchup ];
{
disabledModules = [ # Remove it
inputs.flake-parts.modules.formatter touchup.attr.formatter.enable = false;
];
# perSystem = ...; etc # ... or only have it for listed systems
} touchup.attr.formatter.any.enable = lib.mkDefault false;
touchup.attr.formatter.attr.x86_64-linux.enable = true;
touchup.attr.formatter.attr.aarch64-darwin.enable = true;
'') '')
config.allSystems); config.allSystems);