Skip to content

Enable gosec security checks#974

Merged
jfantinhardesty merged 1 commit into
mainfrom
bugfix/fix-simple-gosec-issues
Jul 16, 2026
Merged

Enable gosec security checks#974
jfantinhardesty merged 1 commit into
mainfrom
bugfix/fix-simple-gosec-issues

Conversation

@jfantinhardesty

Copy link
Copy Markdown
Contributor

What type of Pull Request is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Describe your changes in brief

Add gosec security linter and implement fixes. This PR ignores G115 to be dealt with later due to the number of changes that would need to be made.

Checklist

  • Tested locally
  • Added new dependencies
  • Updated documentation
  • Added tests

Related Issues

  • Related Issue #
  • Closes #

@foodprocessor foodprocessor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🥳

@jfantinhardesty
jfantinhardesty merged commit 6b282d7 into main Jul 16, 2026
32 of 36 checks passed
@jfantinhardesty
jfantinhardesty deleted the bugfix/fix-simple-gosec-issues branch July 16, 2026 21:28
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.

2 participants