Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add extraDebug, lint #134

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add extraDebug, lint #134

wants to merge 1 commit into from

Conversation

roberth
Copy link
Member

@roberth roberth commented Apr 10, 2023

      Diagnostic messages that a flake author may or may not care about.

      For example, a module might detect that it's used in a weird way, but
      not be sure whether that's a mistake or not. Emitting a warning would
      be too much, but with this option, the author can still find the
      detected problem, by enabling [`debug`](#opt-debug) and querying
      the `debug.lint.messages` flake attribute in `nix repl`.

      This feature is not gated by an enable option, as performance does not
      suffer from an unevaluated option.

      There's also no option to upgrade to warnings, because that would make
      evaluation dependent on rather many options, even if the caller only
      needs one specific unrelated thing from the flake.

      A more complex interface could attach the warnings to specific flake
      attribute paths, but that's not implemented for now.

@roberth roberth marked this pull request as draft April 12, 2023 12:08
shymega pushed a commit to shymega/flake-parts that referenced this pull request Aug 18, 2024
shymega pushed a commit to shymega/flake-parts that referenced this pull request Aug 18, 2024
…s-ci#135)

golangci-lint requires all the specified files in its argument to be in
the same directory (Go module). This behavior doesn't fit into treefmt's
model.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant