search

numtide / treefmt-nix

treefmt nix configuration
https://numtide.github.io/treefmt/
MIT License
249 stars 78 forks source link

fixup! fix: only eval enable option if visible (#214) #218

Closed zimbatm closed 2 months ago

zimbatm commented 2 months ago

Turns out not all options have the visible attribute.

Fixes https://github.com/numtide/treefmt-nix/pull/214#issuecomment-2282188773

/cc @phanirithvij

phanirithvij commented 2 months ago

Works with treefmt-nix.url = "github:numtide/treefmt-nix/fix-214";

Apologies for taking up your time, seems like an issue I could've fixed had I been a bit familiar with nix.

phanirithvij commented 2 months ago

I think you may safely merge since all the tests are passing, before more people encounter this issue #217. I can confirm it works.

lxsameer commented 2 months ago

the main branch is already broken. could you please merge the fix

zimbatm commented 2 months ago

thanks @phanirithvij