Skip to content
This repository has been archived by the owner on Feb 15, 2024. It is now read-only.

Replace deprecated linters #219

Merged
merged 1 commit into from Apr 9, 2021
Merged

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Apr 9, 2021

  • replace deprecated maligned linter with
    govet: fieldalignment
  • replace deprecated scopelint linter with
    exportloopref

fixes GH-218

- replace deprecated `maligned` linter with
  `govet: fieldalignment`
- replace deprecated `scopelint` linter with
  `exportloopref`

refs GH-218
@atc0005 atc0005 added bug Something isn't working linting labels Apr 9, 2021
@atc0005 atc0005 added this to the Next Release milestone Apr 9, 2021
@atc0005 atc0005 self-assigned this Apr 9, 2021
@atc0005
Copy link
Owner Author

atc0005 commented Apr 9, 2021

CI failures are expected due to golangci-lint config file changes.

@atc0005 atc0005 merged commit c33c2f1 into master Apr 9, 2021
@atc0005 atc0005 deleted the i218-replace-deprecated-linters branch April 9, 2021 10:40
atc0005 added a commit that referenced this pull request Apr 15, 2021
Disable govet:fieldalignment, re-enable deprecated maligned
linter until the Go team offers more control over the types
of checks provided by the fieldalignment linter or
golangci-lint does so.

refs GH-218
refs GH-219
refs GH-220
refs atc0005/go-ci#302
atc0005 added a commit that referenced this pull request Apr 15, 2021
Disable govet:fieldalignment, re-enable deprecated maligned
linter until the Go team offers more control over the types
of checks provided by the fieldalignment linter or
golangci-lint does so.

- refs GH-218
- refs GH-219
- refs GH-220
- refs atc0005/go-ci#302
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working linting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace deprecated linters: maligned, scopelint
1 participant