treefmt-nix icon indicating copy to clipboard operation
treefmt-nix copied to clipboard

add actionlint

Open katexochen opened this issue 1 year ago • 1 comments

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 avatar Jan 05 '24 17:01 katexochen

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

katexochen avatar Jan 13 '24 09:01 katexochen

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

zimbatm avatar May 30 '24 12:05 zimbatm

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

Nice, looking forward to v2. :)

katexochen avatar May 31 '24 18:05 katexochen

v2 was released.

Mic92 avatar Jun 30 '24 12:06 Mic92

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

zimbatm avatar Jun 30 '24 14:06 zimbatm

This now also happened.

Mic92 avatar Jul 01 '24 15:07 Mic92

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

katexochen avatar Jul 01 '24 15:07 katexochen