Add support for `nickel format`

numtide / treefmt-nix

treefmt nix configuration

https://numtide.github.io/treefmt/

MIT License

228 stars 68 forks source link

Add support for `nickel format` #125

Closed gkze closed 10 months ago

gkze commented 11 months ago

Have to wrap nickel executable because it only accepts one file at a time... I do want to make a PR to Nickel & Topiary to accept multiple file / dir args but want to have something working for now until I do that

gkze commented 11 months ago

🤔 not sure what's up with the (seemingly?) unrelated CI failures...

zimbatm commented 11 months ago

Hey thanks. Make sure to also run the ./examples.sh and treefmt to generate the examples and update the README.md

zimbatm commented 11 months ago

there is also an unrelated error dues to terraform switching to BSL (fixed in main)

gkze commented 10 months ago

(sorry for the delay) I tried running examples.sh but am getting various errors - ktlint not having meta.mainProgram and muon-0.2.0 being marked broken. What should we do?

gkze commented 10 months ago

Ty!

zimbatm commented 10 months ago

I had to move the wrapper out of the options. The default value was depending on config, and that doesn't work (the order of evaluation is imports > options > config).

gkze commented 10 months ago

That's right - thank you for the help! 🙇🏻