search

numtide / treefmt-nix

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

add actionlint #146

Closed katexochen closed 2 months ago

katexochen commented 8 months ago

I would like to add actionlint (which is more a linter than a formatter, but I saw that shellcheck seems to be okay, so actionlint should fit, too).

However, I'm facing some problems with the include statements. For some reason, files in the .github directory are not included/linted.

I have provided a test commit that activates actionlint and includes two files that should throw errors, one in .github, the other in github. Running nix fmt, this will only print errors for the non-hidden dir.

@zimbatm is treefmt in general not capable of formatting hidden files?

katexochen commented 7 months ago

I found https://github.com/numtide/treefmt/issues/153

zimbatm commented 3 months ago

this is better handled by treefmt2, since it can now use git ls-files as a backend

katexochen commented 3 months ago

this is better handled by treefmt2, since it can now use git ls-files as a backend

Nice, looking forward to v2. :)

Mic92 commented 2 months ago

v2 was released.

zimbatm commented 2 months ago

Waiting for nixos-unstable to become available: https://nixpk.gs/pr-tracker.html?pr=323172

Mic92 commented 2 months ago

This now also happened.

katexochen commented 2 months ago

see https://github.com/numtide/treefmt-nix/pull/192 :)