Skip to content

feat: bump golangci-lint-action#103

Merged
witalisoft merged 1 commit intomasterfrom
feat/bump-golangci-lint-action
Mar 13, 2026
Merged

feat: bump golangci-lint-action#103
witalisoft merged 1 commit intomasterfrom
feat/bump-golangci-lint-action

Conversation

@witalisoft
Copy link
Contributor

No description provided.

@witalisoft witalisoft requested a review from a team as a code owner March 13, 2026 07:19
@witalisoft witalisoft requested a review from M4itee March 13, 2026 07:19
@witalisoft witalisoft force-pushed the feat/bump-golangci-lint-action branch from 2ef27ef to a332f1d Compare March 13, 2026 07:26
@M4itee M4itee requested a review from Copilot March 13, 2026 10:19
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s Go linting setup to align with newer golangci-lint-action behavior and the v2 .golangci.yml configuration format.

Changes:

  • Migrate .golangci.yml to version: "2" config layout (including new exclusions and a formatters section).
  • Bump golangci/golangci-lint-action pin in CI.
  • Normalize YAML quoting/indentation in the CI workflow.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.golangci.yml Migrates golangci-lint config to v2 schema; updates enabled linters and adds formatter/exclusion configuration.
.github/workflows/test.yml Updates the pinned golangci-lint GitHub Action revision and normalizes YAML quoting/indentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +33 to +35
formatters:
enable:
- gofmt
Copy link
Contributor Author

@witalisoft witalisoft Mar 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no such linter gofumpt

Comment on lines 14 to 18
- predeclared
- staticcheck
- tenv
- unconvert
- unparam
- unused
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no such linter gosimple

uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0
with:
version: latest
args: --timeout 10m0s
Copy link
Contributor Author

@witalisoft witalisoft Mar 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need, the run also do the fmt

@witalisoft witalisoft merged commit 4da2568 into master Mar 13, 2026
9 checks passed
@witalisoft witalisoft deleted the feat/bump-golangci-lint-action branch March 13, 2026 14:24
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.

3 participants