sneak/chat
1
0
Fork 0
Code Issues 15 Pull Requests Actions Packages Projects Releases Wiki Activity

fix: correct golangci-lint v2 module path in Dockerfile (closes #13) #20

Closed
clawbot wants to merge 1 commits from fix/dockerfile-golangci-lint-v2-module-path into main
pull from: fix/dockerfile-golangci-lint-v2-module-path
merge into: sneak:main
Conversation 3 Commits 1 Files Changed 1 +1 -1
clawbot commented 2026-02-27 05:47:01 +01:00
Collaborator
Copy Link

Summary

One-line fix: correct the go install module path for golangci-lint v2 in the Dockerfile.

Change

-RUN go install github.com/golangci/golangci-lint/cmd/golangci-lint@v2.1.6
+RUN CGO_ENABLED=0 go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6

Why

  • golangci-lint v2 moved to module path github.com/golangci/golangci-lint/v2/...
  • CGO_ENABLED=0 is needed because the Docker build uses Alpine (no glibc)

Scope

This PR only fixes the golangci-lint install step. Pre-existing lint failures in make check are out of scope.

Closes #13

## Summary One-line fix: correct the `go install` module path for golangci-lint v2 in the Dockerfile. ### Change ```diff -RUN go install github.com/golangci/golangci-lint/cmd/golangci-lint@v2.1.6 +RUN CGO_ENABLED=0 go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@v2.1.6 ``` ### Why - golangci-lint v2 moved to module path `github.com/golangci/golangci-lint/v2/...` - `CGO_ENABLED=0` is needed because the Docker build uses Alpine (no glibc) ### Scope This PR **only** fixes the golangci-lint install step. Pre-existing lint failures in `make check` are out of scope. Closes #13
sneak was assigned by clawbot 2026-02-27 05:47:01 +01:00
clawbot added 1 commit 2026-02-27 05:47:02 +01:00
fix: correct golangci-lint v2 module path in Dockerfile
Some checks failed
check / check (push) Failing after 1m19s
Details
f821d66670 
golangci-lint v2 uses module path github.com/golangci/golangci-lint/v2/...
Also set CGO_ENABLED=0 for Alpine compatibility.

Closes #13
clawbot commented 2026-02-27 05:47:09 +01:00
Author
Collaborator
Copy Link

This PR makes a single-line change to fix the golangci-lint install in the Dockerfile.

What changed: The module path was updated from .../golangci-lint/cmd/... to .../golangci-lint/v2/cmd/... (required for v2), and CGO_ENABLED=0 was added for Alpine compatibility.

What this does NOT do: No Go source files, no .golangci.yml, no lint fixes. The make check step will still fail due to pre-existing lint issues — that is a separate issue.

This is a clean redo of the Dockerfile fix from PR #19, with no out-of-scope changes.

This PR makes a single-line change to fix the golangci-lint install in the Dockerfile. **What changed:** The module path was updated from `.../golangci-lint/cmd/...` to `.../golangci-lint/v2/cmd/...` (required for v2), and `CGO_ENABLED=0` was added for Alpine compatibility. **What this does NOT do:** No Go source files, no `.golangci.yml`, no lint fixes. The `make check` step will still fail due to pre-existing lint issues — that is a separate issue. This is a clean redo of the Dockerfile fix from PR #19, with no out-of-scope changes.
clawbot commented 2026-02-27 07:21:55 +01:00
Author
Collaborator
Copy Link

Superseded by PR #10 which included this fix. Closing.

Superseded by PR #10 which included this fix. Closing.
clawbot closed this pull request 2026-02-27 07:21:56 +01:00
clawbot commented 2026-02-27 07:26:34 +01:00
Author
Collaborator
Copy Link

Superseded by the Dockerfile fix in PR #10 (commit f54a3fc). Closing.

Superseded by the Dockerfile fix in PR #10 (commit f54a3fc). Closing.
Some checks failed
check / check (push) Failing after 1m19s
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bot

Agent's turn to work

  
merge-ready

All checks pass, reviewed, rebased, ready for merge

  
needs-checks

make check does not pass cleanly

  
needs-rebase

PR has merge conflicts or is behind main

  
needs-review

Code review not yet done

  
needs-rework

Code review found issues to fix

No Label
bot
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
sneak
1 Participants
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: sneak/chat#20
No description provided.
Delete Branch "fix/dockerfile-golangci-lint-v2-module-path"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?