Skip to content

[Codegen] Update lint module for tests#1191

Merged
jamietanna merged 1 commit into
oapi-codegen:masterfrom
eugenio-bolt:eugenio/update-lint-module
Aug 18, 2023
Merged

[Codegen] Update lint module for tests#1191
jamietanna merged 1 commit into
oapi-codegen:masterfrom
eugenio-bolt:eugenio/update-lint-module

Conversation

@eugenio-bolt

@eugenio-bolt eugenio-bolt commented Aug 11, 2023

Copy link
Copy Markdown
Contributor
  • Replace github.com/golangci/lint-1 with golang.org/x/lint
  • Run go mod tidy
  • This change is required for users of oapi-codegen that use bazel, as they can encounter errors when dependencies do not have a go.mod file, as is the case with github.com/golangci/lint-1

@jamietanna

Copy link
Copy Markdown
Member

Thanks @eugenio-bolt! I see that the version we're replacing is an older fork of the /x/ repo, so very much makes sense to do anyway!

@jamietanna jamietanna merged commit 4887803 into oapi-codegen:master Aug 18, 2023
debuggerpk pushed a commit to breuHQ/oapi-codegen that referenced this pull request Sep 18, 2023
The `github.com/golangci/lint-1` module has been unmaintained for
several years, and is missing several features and bugfixes which leads
to this breaking with Bazel builds.
danicc097 pushed a commit to danicc097/oapi-codegen that referenced this pull request Aug 31, 2024
The `github.com/golangci/lint-1` module has been unmaintained for
several years, and is missing several features and bugfixes which leads
to this breaking with Bazel builds.
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.

2 participants