Skip to content

upgrade golangci-lint version#173

Merged
dhth merged 1 commit into
mainfrom
upgrade-golangci-lint-version
Jul 17, 2026
Merged

upgrade golangci-lint version#173
dhth merged 1 commit into
mainfrom
upgrade-golangci-lint-version

Conversation

@dhth

@dhth dhth commented Jul 17, 2026

Copy link
Copy Markdown
Owner

No description provided.

@dhth
dhth merged commit b935a51 into main Jul 17, 2026
12 of 13 checks passed
@dhth
dhth deleted the upgrade-golangci-lint-version branch July 17, 2026 15:41
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0f53272f-6984-4c1d-bb04-5e3d82aa8b2d

📥 Commits

Reviewing files that changed from the base of the PR and between b6ae21c and b2e3eb6.

📒 Files selected for processing (16)
  • .github/workflows/main.yml
  • .github/workflows/pr.yml
  • .golangci.yml
  • cmd/root.go
  • internal/persistence/migrations.go
  • internal/persistence/queries.go
  • internal/types/date_helpers.go
  • internal/ui/handle.go
  • internal/ui/help.go
  • internal/ui/initial.go
  • internal/ui/log.go
  • internal/ui/report.go
  • internal/ui/stats.go
  • internal/ui/theme/builtin.go
  • internal/ui/update.go
  • internal/ui/view.go

📝 Walkthrough

Walkthrough

The lint workflows now install Go from go.mod and run golangci-lint v2.12 directly. Linter exclusions were added for a color literal and test files. The today period is centralized in types.TimePeriodToday and reused by command and date-range logic. Several persistence and UI calls were reformatted without changing their behavior.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive No pull request description was provided, so its relation to the changeset can't be evaluated. Add a brief description of the linting and workflow changes to make the PR easier to review.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the main change: bumping golangci-lint in CI.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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