Skip to content

Set up linter#539

Merged
natefinch merged 9 commits intomasterfrom
nf/linter
Mar 12, 2026
Merged

Set up linter#539
natefinch merged 9 commits intomasterfrom
nf/linter

Conversation

@natefinch
Copy link
Copy Markdown
Member

Add .golangci.toml configuration file for linter settings.

Add .golangci.toml configuration file for linter settings.
perrito666
perrito666 previously approved these changes Mar 8, 2026
@ccoVeille
Copy link
Copy Markdown

Using a .toml file on a Go project sounds a bit odd to me

I would have used .yaml

But why not

@natefinch
Copy link
Copy Markdown
Member Author

natefinch commented Mar 8, 2026

I hate yaml. It's error prone. Significant white space is just bad.

@natefinch natefinch marked this pull request as draft March 9, 2026 15:15
@natefinch natefinch marked this pull request as ready for review March 12, 2026 11:54
@natefinch natefinch merged commit 3594347 into master Mar 12, 2026
44 checks passed
@natefinch natefinch deleted the nf/linter branch March 12, 2026 11:54
This was referenced Mar 29, 2026
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