-
Notifications
You must be signed in to change notification settings - Fork 89
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
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: