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

docs: exhaustivestruct example explanation #1985

Merged
merged 1 commit into from
May 15, 2021

Conversation

butuzov
Copy link
Member

@butuzov butuzov commented May 15, 2021

Comment add context on struct-patterns patterns, so it's not unexpected for users that patterns behave in next way:

  1. e.g. *.Text can match struct Test if golangci-lint launched against the file
    golangcli-lint run example/example.go

  2. e.g. *.Text do not match struct Test if golangci-lint launched against package
    golangcli-lint run example

@butuzov butuzov requested a review from ldez May 15, 2021 13:04
@ldez ldez added area: docs enhancement New feature or improvement labels May 15, 2021
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment add context on `struct-patterns` patterns, so its not unexpected
for users that patterns behaves in next way:

1. e.g. `*.Text` can match struct Test if golangci-lint launched against
   the file
   `golangcli-lint run example/example.go`

1. e.g. `*.Text` do not match struct Test if golangci-lint launched
   against package
   `golangcli-lint run example`
@ldez ldez merged commit 42a3299 into golangci:master May 15, 2021
@butuzov butuzov deleted the Fix-Exhaustivestruct branch May 16, 2021 01:14
SeigeC pushed a commit to SeigeC/golangci-lint that referenced this pull request Apr 4, 2023
@ldez ldez added this to the v1.41 milestone Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: docs enhancement New feature or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants