Skip to content

Conversation

pmlanger
Copy link
Contributor

@pmlanger pmlanger commented Sep 8, 2025

https://typeform.atlassian.net/browse/PLT-2300

Make the configuration file location of golangci-lint (golangci.yaml) configurable.

This is a preparation for the move to golangci-lint v2. The configuration files between v1 and v2 are not compatible, which means that once we move the golangci-lint executable to 2.x, a single location for the file is not sufficient, as some repos will be on v1, and others on v2.

The defaults are the hardcoded values from today, with the exception of ref, which is set to 1.15.0 (latest tag with a v1 configuration file).

Tested for backwards compatibility: https://github.com/Typeform/event-dispatcher/actions/runs/17552732900/job/49848910468?pr=605

@pmlanger pmlanger force-pushed the PLT-2300-linter-config branch from 7db43f1 to f9e69dd Compare September 8, 2025 11:12
@pmlanger pmlanger changed the title feat: configurable golangci-lint config location feat: configurable golangci-lint config Sep 8, 2025
@pmlanger pmlanger changed the title feat: configurable golangci-lint config feat: dynamic golangci-lint versions and configuration Sep 8, 2025
@pmlanger pmlanger force-pushed the PLT-2300-linter-config branch from 85f02bd to 89147de Compare September 8, 2025 12:44
@pmlanger pmlanger marked this pull request as ready for review September 8, 2025 13:44
@pmlanger pmlanger requested a review from a team as a code owner September 8, 2025 13:44
Copy link

gitstream-cm bot commented Sep 8, 2025

🥷 Code experts: no user but you matched threshold 10

pmlanger has most 👩‍💻 activity in the files.
trapped has most 🧠 knowledge in the files.

See details

.github/workflows/go-lint-workflow.yaml

Activity based on git-commit:

pmlanger
SEP 2 additions & 2 deletions
AUG 1 additions & 1 deletions
JUL
JUN
MAY
APR

Knowledge based on git-blame:
trapped: 10%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

@pmlanger pmlanger merged commit 8c03647 into main Sep 8, 2025
8 checks passed
@pmlanger pmlanger deleted the PLT-2300-linter-config branch September 8, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants