[PR #387] [CLOSED] fix: update golangci-lint to v1.64.8 to resolve linter errors #400

Closed
opened 2026-03-13 16:21:45 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/acme-dns/acme-dns/pull/387
Author: @michvllni
Created: 6/18/2025
Status: Closed

Base: masterHead: fix/linter-update


📝 Commits (1)

  • 2d17bfd fix: update golangci-lint to v1.64.8 to resolve linter errors

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 .github/workflows/golangci-lint.yml (+1 -1)

📄 Description

In the current version (v1.60) the linter action fails with errors:

Running [/home/runner/golangci-lint-1.60.3-linux-amd64/golangci-lint config path] in [/home/runner/work/acme-dns/acme-dns] ...
  Running [/home/runner/golangci-lint-1.60.3-linux-amd64/golangci-lint run  --timeout=10m] in [/home/runner/work/acme-dns/acme-dns] ...
  Error: validation.go:4:2: could not import unicode/utf8 (-: could not load export data: internal error in importing "unicode/utf8" (unsupported version: 2); please report an issue) (typecheck)
  	"unicode/utf8"
  	^
  Error: api_test.go:182:17: undefined: sqlmock (typecheck)
  	db, mock, _ := sqlmock.New()
  	               ^
  Error: api_test.go:[31](https://github.com/michvllni/acme-dns/actions/runs/15729056033/job/44325629202#step:4:33)1:17: undefined: sqlmock (typecheck)
  	db, mock, _ := sqlmock.New()
  	               ^
  Error: db_test.go:31:2: undefined: testdb (typecheck)
  	testdb.SetExecWithArgsFunc(func(query string, args []driver.Value) (result driver.Result, err error) {
  	^
  Error: db_test.go:[34](https://github.com/michvllni/acme-dns/actions/runs/15729056033/job/44325629202#step:4:36):8: undefined: testdb (typecheck)
  	defer testdb.Reset()
  	      ^
  Error: db_test.go:114:8: undefined: testdb (typecheck)
  	defer testdb.Reset()
  	      ^
  
  Error: issues found
  Ran golangci-lint in 51617ms

Updating to 1.64.8 (latest 1.X) resolves this.

We should make plans to migrate to version 2, but at the moment it threw to many errors for me to safely resolve them without further discussion.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/acme-dns/acme-dns/pull/387 **Author:** [@michvllni](https://github.com/michvllni) **Created:** 6/18/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix/linter-update` --- ### 📝 Commits (1) - [`2d17bfd`](https://github.com/acme-dns/acme-dns/commit/2d17bfd8ff021d78444dafdd0b26841c9c1eaa04) fix: update golangci-lint to v1.64.8 to resolve linter errors ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golangci-lint.yml` (+1 -1) </details> ### 📄 Description In the current version (v1.60) the linter action fails with errors: ``` Running [/home/runner/golangci-lint-1.60.3-linux-amd64/golangci-lint config path] in [/home/runner/work/acme-dns/acme-dns] ... Running [/home/runner/golangci-lint-1.60.3-linux-amd64/golangci-lint run --timeout=10m] in [/home/runner/work/acme-dns/acme-dns] ... Error: validation.go:4:2: could not import unicode/utf8 (-: could not load export data: internal error in importing "unicode/utf8" (unsupported version: 2); please report an issue) (typecheck) "unicode/utf8" ^ Error: api_test.go:182:17: undefined: sqlmock (typecheck) db, mock, _ := sqlmock.New() ^ Error: api_test.go:[31](https://github.com/michvllni/acme-dns/actions/runs/15729056033/job/44325629202#step:4:33)1:17: undefined: sqlmock (typecheck) db, mock, _ := sqlmock.New() ^ Error: db_test.go:31:2: undefined: testdb (typecheck) testdb.SetExecWithArgsFunc(func(query string, args []driver.Value) (result driver.Result, err error) { ^ Error: db_test.go:[34](https://github.com/michvllni/acme-dns/actions/runs/15729056033/job/44325629202#step:4:36):8: undefined: testdb (typecheck) defer testdb.Reset() ^ Error: db_test.go:114:8: undefined: testdb (typecheck) defer testdb.Reset() ^ Error: issues found Ran golangci-lint in 51617ms ``` Updating to 1.64.8 (latest 1.X) resolves this. We should make plans to migrate to version 2, but at the moment it threw to many errors for me to safely resolve them without further discussion. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-13 16:21:45 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/acme-dns#400
No description provided.