Skip to content

fix: enable more go-critic rules#2582

Merged
simar7 merged 4 commits intoaquasecurity:mainfrom
mmorel-35:go-critic
Jun 5, 2025
Merged

fix: enable more go-critic rules#2582
simar7 merged 4 commits intoaquasecurity:mainfrom
mmorel-35:go-critic

Conversation

@mmorel-35
Copy link
Copy Markdown
Contributor

@mmorel-35 mmorel-35 commented May 31, 2025

Description

This enables the following go-critic rules:

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@mmorel-35 mmorel-35 requested a review from simar7 as a code owner May 31, 2025 10:54
@mmorel-35 mmorel-35 changed the title Go critic fix: enable more go-critic rules May 31, 2025
@github-actions github-actions bot added the bug label May 31, 2025
@mmorel-35 mmorel-35 force-pushed the go-critic branch 2 times, most recently from 88900b9 to 012f8c6 Compare June 4, 2025 07:22
@simar7 simar7 self-requested a review June 5, 2025 04:20
@simar7 simar7 merged commit ce7661e into aquasecurity:main Jun 5, 2025
9 checks passed
@mmorel-35 mmorel-35 deleted the go-critic branch June 5, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants