refactor: run go fix #107

noe.trevino · 2026-02-18T02:55:06-06:00

noe.trevino commented

2026-02-18 02:55:06 -06:00

go fix ./...
literally the best thing ever

Checklist

Read CONTRIBUTING.md
Ran tests (go test ./... -race) locally
Formatted and linted with gofmt -w . and golangci-lint run ./... respectively
Added tests if necessary (significant changes/complex logic)
Updated documentation if applicable
Updated README.md if applicable

Summary

Uses go fix to implement modernization 'automagically'

- go fix ./... - literally the best thing ever ## Checklist - [x] Read `CONTRIBUTING.md` - [x] Ran tests (`go test ./... -race`) locally - [x] Formatted and linted with `gofmt -w .` and `golangci-lint run ./...` respectively - [x] Added tests if necessary (significant changes/complex logic) - [x] Updated documentation if applicable - [x] Updated `README.md` if applicable ## Summary  Uses go fix to implement modernization 'automagically'

claude[bot] (Migrated from github.com) reviewed

2026-02-18 02:59:04 -06:00

internal/testing/fixtures/db.go

					
				@ -442,2 +442,3 @@

							result.WriteString(sep)

						}

						result += s

						result.WriteString(s)

The joinStrings helper still exists after this PR and now uses strings.Builder. This function is equivalent to strings.Join(strs, sep) from the standard library, which would eliminate the helper entirely. go fix won't catch this, but it's a clear DRY/simplification opportunity that was touched in this PR.

The `joinStrings` helper still exists after this PR and now uses `strings.Builder`. This function is equivalent to `strings.Join(strs, sep)` from the standard library, which would eliminate the helper entirely. `go fix` won't catch this, but it's a clear DRY/simplification opportunity that was touched in this PR.

Sign in to join this conversation.

No reviewers