Skip to content

Exclude generated code from being processed by gosec#138

Merged
falfaroc merged 2 commits intomainfrom
exclude-generated-code-checks
Mar 19, 2025
Merged

Exclude generated code from being processed by gosec#138
falfaroc merged 2 commits intomainfrom
exclude-generated-code-checks

Conversation

@falfaroc
Copy link
Copy Markdown
Contributor

@falfaroc falfaroc commented Mar 19, 2025

Description

Update the gosec action to include the -exclude-generated flag. This will bypass checking any file that has been generated by some generator.

See: https://github.com/securego/gosec?tab=readme-ov-file#excluding-generated-files

Issues

List the issues impacted by this PR:

Issue ID
https://github.com/dell/csm/issues/1707

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

@falfaroc falfaroc marked this pull request as draft March 19, 2025 13:22
@falfaroc falfaroc force-pushed the exclude-generated-code-checks branch from cf50cd0 to e496a75 Compare March 19, 2025 13:29
@falfaroc falfaroc marked this pull request as ready for review March 19, 2025 13:30
Comment thread gosec-runner/entrypoint.sh Outdated
@falfaroc falfaroc merged commit 401dfad into main Mar 19, 2025
2 checks passed
@falfaroc falfaroc deleted the exclude-generated-code-checks branch March 19, 2025 14:13
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