Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(gh): update golangci-lint workflow #554

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

tenthirtyam
Copy link
Collaborator

Summary of Pull Request

  • Updates golangci-lint configuration.
  • Updates the GitHub Action for golangci-lint.
  • Pin GitHub Actions to release commit SHA.

Type of Pull Request

  • This is a bug fix.
  • This is an enhancement or feature.
  • This is a code style/formatting update.
  • This is a documentation update.
  • This is a refactoring update.
  • This is a chore update
  • This is something else.
    Please describe:

Related to Existing Issues

Test and Documentation Coverage

For bug fixes or features:

  • Tests have been completed.
  • Documentation has been added/updated.

BEFORE:

golangci-lint run
WARN [config_reader] The configuration option `linters.errcheck.ignore` is deprecated, please use `linters.errcheck.exclude-functions`. 

@tenthirtyam tenthirtyam added chore Chore github-actions GitHub Actions labels Nov 19, 2024
@tenthirtyam tenthirtyam requested a review from frodenas November 19, 2024 17:18
@tenthirtyam tenthirtyam self-assigned this Nov 19, 2024
- Updates `golangci-lint` configuration.
- Updates the GitHub Action for `golangci-lint`.
- Pin GitHub Actions to release commit SHA.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam force-pushed the chore(gh)/update-golangci-lint branch from bc0e472 to 1535229 Compare November 19, 2024 17:23
@tenthirtyam tenthirtyam added this to the .next milestone Dec 2, 2024
@tenthirtyam tenthirtyam merged commit 27c4c77 into main Dec 2, 2024
5 checks passed
@tenthirtyam tenthirtyam deleted the chore(gh)/update-golangci-lint branch December 2, 2024 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore github-actions GitHub Actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant