inappropriate unused_binding diagnostic with named attribute set argument (@) #73
Comments
I should clarify: if the whole attrset is passed into a function, rather than another attr ( |
I should also explain, the reason those params are listed is to tell nix to add them as inputs from the registry. |
|
Confirmed fixed with flake inputs. I do feel like this could still be an issue in general with named attrset arguments that get passed into a function defined elsewhere, but I don't have a specific instance in mind so I'm happy with this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is a general issue, but the common use-case where I'm hitting it is in a flake when utilizing the registry to avoid specifying an explicit input URL:
inputs.nixos-hardware is not used directly in the function body, but in an expression in an included file which is passed
inputs(same forhome-managerandhelix(src). Basically, since nil doesn't analyze into included files, it should be enough if the named attribute set argument (here, inputs) is referenced in the function body. Does that make sense?Also, thank you so much for working on nil. It has greatly improved my daily experience of working on Nix code, and with each new analysis feature it gets better. ❤️
The text was updated successfully, but these errors were encountered: