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.

Sign up for GitHub

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

Jump to bottom

.golangci.yml: add intrange linter #3572

Merged
merged 2 commits into from
Sep 5, 2024
Merged

.golangci.yml: add intrange linter #3572

merged 2 commits into from
Sep 5, 2024

Commits on Sep 4, 2024

  1. .golangci.yml: add intrange linter 

    It checks that go 1.22's range-over-numbers feature is not skipped. Also,
fix some warnings it found.

Signed-off-by: Pavel Karpy <[email protected]>
    @carpawell
    carpawell committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    32f91dd View commit details
    Browse the repository at this point in the history

  2. mpt: refactor lcp to be possible to use range 

    It took some time to understand why changing a regular `for` to a `range`
one leads to behavior changes; let it be more clear and explicit. Also, a
correct code is always better than a correct code with `nolint`.

Signed-off-by: Pavel Karpy <[email protected]>
    @carpawell
    carpawell committed Sep 4, 2024
    Configuration menu
    Copy the full SHA
    f8549a4 View commit details
    Browse the repository at this point in the history