Skip to content

Conversation

@swift1337
Copy link
Collaborator

@swift1337 swift1337 commented Dec 23, 2025

Apply new linting rules to the codebase.


Note

Overview

Applies stricter linting and formatting across the repo and adjusts code accordingly; behavior remains the same.

  • Updates .golangci.yml: adds golines, reorders gci sections with project prefix, sets generated: strict, and refines exclusion/rule presets (e.g., scoped staticcheck, revive var-naming, errcheck, gosec patterns)
  • Widespread lint-driven refactors: reorder imports, wrap long args, convert unused params to _, add //nolint where appropriate, and improve structured logging formatting
  • Security/IO tweaks: annotate intentional file perms (MkdirAll/OpenFile/WriteFile) and close/sync handling in logging/results utilities
  • Minor test/log output adjustments and small helper signature cleanups to satisfy linters

Written by Cursor Bugbot for commit 135f0be. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@swift1337 swift1337 force-pushed the feat/lint branch 2 times, most recently from 934d4e6 to 11342a7 Compare December 23, 2025 12:32
@swift1337 swift1337 self-assigned this Dec 23, 2025
@aljo242
Copy link
Collaborator

aljo242 commented Dec 23, 2025

can just // nolint this error

@swift1337 swift1337 merged commit ab16e12 into feat/improvements Dec 23, 2025
4 checks passed
@swift1337 swift1337 deleted the feat/lint branch December 23, 2025 19:32
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