akshay
akshay
Ah, flake attributes such as `inputs`etc. are a little special cased. They are not allowed to perform any evaluations to produce the `outputs` attribute, for example. Currently `statix` does not...
i would say this is a formatting preference, and should be handled by formatting tools, rather than a linter, cc'ing some contributors for their opinions cc @antifuchs @chenrui333 @NobbZ
Thanks for the PR and sorry for the tardy reply! Just a couple of comments: - could you add unit tests for the lint? the process is as follows: *...
Hmm, this is a tall ask :sweat_smile:. I think it partly requires rethinking how the whole lint framework behaves altogether. Currently lints are expected to be as "local" as possible....
Hey, thanks for opening an issue (and thanks for your work on nix-linter)! Some of these lints are definitely on the roadmap.
@tadfisher could you describe the `lock` task used in the gist?
would you have a few examples of this scenario? perhaps there are some heuristics to determine and avoid such cases.
thanks for all your PRs over the past couple of weeks, my time is a little limited at the moment, I'll take a look as soon as I am free!
A `Helper` shouldn't have to have all three trait implementations. Maybe we can have something like `set_hinter`, `set_highlighter` and `set_completer`? Right now, its a little roundabout to implement just one...
this should be applicable to `'a'..='z'.contains(&c)` as well.