-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
dprint is inconsistent #278
Comments
Also a nix flake check with dprint, with plugins configured, is failing.
$ nix-shell -p nix-info --run "nix-info -m"
- system: `"x86_64-linux"`
- host os: `Linux 6.6.66, NixOS, 25.05 (Warbler), 25.05pre-git`
- multi-user?: `yes`
- sandbox: `yes`
- version: `nix-env (Nix) 2.24.11`
$ nix config show | grep sandbox
sandbox = true
sandbox-build-dir = /build
sandbox-dev-shm-size = 50%
sandbox-fallback = false whereas a |
For the second issues, try replacing the |
Thanks, that is what I ended up doing. but I don't know how to keep them up-to-date. I'm thinking it is best to define the plugins in dprint.json, so that it allows for easy auto updates, even if doing this is imperative. or need to come up with a |
The first thing to do is to assemble the package set somewhere. Probably in nixpkgs since it has all the update infrastructure. |
alright, I will try it out |
feel free to ping me on nixpkgs side if you send a PR. for your first observation, this is a general problem with the current design of the modules. I will see what I can do. |
okay. |
with that pr this is working plugins = map toString (
with pkgs.dprint-plugins; # or dprint.plugins
[
dprint-plugin-json
dprint-plugin-markdown
dprint-plugin-toml
g-plane-pretty_yaml
]
); |
okay, programs.dprint.includes = [ "**/*.md" ]; # works
programs.dprint.settings.includes = [ "**/*.md" ]; # no longer works, it uses ".*" (set as default in dprint.nix) you may close this issue @zimbatm if you think this is done. Thank you for guiding me through the above nixpkgs pr as well as this treewide fix. |
Describe the bug
Expected behavior
Expect the settings API to be consistent.
System information
latest commit 0ce9d14, via flake, nixpkgs-unstable.
The text was updated successfully, but these errors were encountered: