Skip to content

Conversation

@redbeam
Copy link
Contributor

@redbeam redbeam commented Nov 6, 2025

Fixes golangci-lint issues; they're mostly about checking Fprintf for errors and handling errors in deferred functions.

@DeedleFake
Copy link
Owner

DeedleFake commented Nov 7, 2025

Thank you for the pull request.

I do not want all of those extra _, _ = before every single Fprint*() call. I do not agree with golangci-lint on this in the slightest.

The defer changes and the embedded redundancy removal are probably fine. Remove the _, _ and I'll take another look.

@redbeam redbeam force-pushed the fix-golangci-lint-issues branch from 516aae5 to 372c809 Compare November 10, 2025 14:55
@redbeam
Copy link
Contributor Author

redbeam commented Nov 10, 2025

Yeah, I get your point. The other option would be to silence them using //nolint:errcheck, but I'm not sure if that's any better.

Anyway, I dropped the commit.

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