Skip to content

chore: Add golangci lint - #492

Merged
YuKitsune merged 4 commits into
mainfrom
eoin/sf/add-golangci-lint
Oct 15, 2026
Merged

YuKitsune merged 4 commits into
mainfrom
eoin/sf/add-golangci-lint

Conversation

@YuKitsune

@YuKitsune YuKitsune commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Adds a golangci-lint workflow, and addresses all found linting issues.

YuKitsune and others added 3 commits October 15, 2026 11:09
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@YuKitsune
YuKitsune marked this pull request as ready for review October 15, 2026 01:34

@APErebus APErebus 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.

I didn't check every file, but did a skim through and almost all of them are just reordering of imports.

1 minor nit, but I think it's good to merge.

Comment thread README.md Resolved
Co-authored-by: Alastair Pitts <APErebus@users.noreply.github.com>
@YuKitsune
YuKitsune requested a review from APErebus October 15, 2026 05:31

@APErebus APErebus 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.

🚀

@YuKitsune
YuKitsune merged commit 2ee8fb7 into main Oct 15, 2026
5 checks passed
@YuKitsune
YuKitsune deleted the eoin/sf/add-golangci-lint branch October 15, 2026 05:33
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