Disable a formatter for a path

[corentinmusard]

Welcome

• Yes, I've searched similar issues on GitHub and didn't find any.
• I agree to follow this project's Code of Conduct

How did you install golangci-lint?

Brew

Your feature request related to a problem? Please describe

(using version 2.0.1)

Can you make it possible to disable a specific formatter for a give path?

For linters we have

  exclusions:
    rules:
      # Exclude some linters from running on tests files.
      - linters:
          - containedctx
          - dogsled
        path: _test.go

Use case: I want to enable golines everywhere but not on my _test.go files.

Describe the solution you'd like

formatters:
  enable:
    - golines
  exclusions:
    paths:
      - linters:
          - golines
        path: _test.go

Describe alternatives you've considered

Keeping golines disabled

Additional context

No response

Supporter

• I am a sponsor through GitHub or OpenCollective

[boring-cyborg]

Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors.

[ldez]

Hello,

Currently, you cannot disable one formatter for a path, but you can exclude a path (for all formatters):

formatters:
  paths:
    - _test.go

Can you explain why do you want to apply formatters except one on a specific file?

[corentinmusard]

Thank you for the quick response!

We have a lot of long one-liner in our units tests that I want to keep in a single line. Such as:

createdUser := client.User.Create().SetID("dummy_id").SetName("Dummy User").SetEmail("dummy_user@domain.com").SaveX(ctx)

Making them multiline makes our tests much longer and less readable, so I'd like to disable this formatter for test files

[thiagodeev]

Same here. We have this interface, and I would prefer to keep it single-lined.
Something like //noformat:golines would be great for specific pieces of code

[ldez]

The exclusions based on a path are possible, but a directive for formatters will never happen: formatters are not linters, the formatting is applied to raw files, not parsed files.

[thiagodeev]

@ldez got it, thanks