Skip to content

Fix lint and style issues#138

Open
marcus wants to merge 1 commit intomainfrom
codex/lint-fix
Open

Fix lint and style issues#138
marcus wants to merge 1 commit intomainfrom
codex/lint-fix

Conversation

@marcus
Copy link
Copy Markdown
Owner

@marcus marcus commented Apr 25, 2026

Summary

  • applies lint/style cleanup already staged on codex/lint-fix across 24 Go files
  • no additional edits were needed on verification branch codex/lint-fix-automated-cleanup-2
  • opens the delivery PR from codex/lint-fix into main, correcting the prior branch-to-base PR path

Checks

  • gofmt -l $(rg --files -g '*.go')
  • go vet ./...
  • make lint
  • go test ./...
  • git diff --check main..codex/lint-fix

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift


Automated by nightshift

Fix errorlint: use errors.Is for sentinel error comparisons (io.EOF,
scheduler.ErrNotRunning) and errors.As for type assertions (*exec.ExitError)
in provider-calibration, daemon, and all three agent packages.

Fix perfsprint: replace fmt.Sprintf("%d") with strconv.Itoa/FormatInt in
budget, report, status, task, provider tests, tmux, and provider-calibration.
Replace fmt.Errorf with errors.New where no format verbs are used.

Fix intrange: use Go 1.22+ integer range syntax in budget progressBar and
config struct iteration loops.

Fix mirror: replace bytes.HasPrefix([]byte(s), []byte(p)) with
strings.HasPrefix in analysis/report.go; replace b.WriteString(string(x))
with b.Write(x) in report.go.

Fix gofumpt: apply formatting (octal literals, var grouping) in config.go
and daemon.go.

Fix godot: add trailing periods to block comments in init.go and install.go.

Fix wastedassign: use var declaration instead of dead empty-string init in
helpers.go and install.go.

Fix prealloc: pre-allocate slices in github.go, td.go, register.go,
audit.go, and stats.go.

Fix forcetypeassert: add checked type assertions in setup_test.go.

Fix errchkjson: check json.Marshal errors in orchestrator_test.go.

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift
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.

1 participant