keb/lazysecrets
1
0 Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases 5 Wiki Activity

refactor: drop dead code and use the standard library where it already does the job #76

Merged
keb merged 1 commits from refactor/dead-code-stdlib into main 2026-10-15 10:39:38 +02:00
Conversation 0 Commits 1 Files Changed 43
+161 -304
keb commented 2026-10-15 08:55:09 +02:00
Owner
Copy link
Copy source

No behaviour change.

  • Deleted, unused or one-line wrappers: panes.RuneIndexes, Detail.Refresh (SetDescribed does it), SecretValue.Len, Query.Empty, Clipboard.Native, config.Load, get's value, Model.done, providerOf (cachefile.ProviderOf), contextWithTimeout, searchCoverage, startCrawl, showKeybindings, the expiry aliases and daysLeft.
  • cmp.Or instead of the two cmpOr copies; slices.Sorted(maps.Keys(…)), maps.Copy, wg.Go, utf8.RuneCountInString and new(true) instead of loops and helpers that do the same.
  • A container's pane title is set once, not twice; a redundant length check in share links and a no-op assignment in linkhandler.Launch are gone.
  • Tests call ansi.Strip and lastLines directly; gofmt -s on links.go.

🤖 Generated with Claude Code

No behaviour change. - Deleted, unused or one-line wrappers: panes.RuneIndexes, Detail.Refresh (SetDescribed does it), SecretValue.Len, Query.Empty, Clipboard.Native, config.Load, get's value, Model.done, providerOf (cachefile.ProviderOf), contextWithTimeout, searchCoverage, startCrawl, showKeybindings, the expiry aliases and daysLeft. - cmp.Or instead of the two cmpOr copies; slices.Sorted(maps.Keys(…)), maps.Copy, wg.Go, utf8.RuneCountInString and new(true) instead of loops and helpers that do the same. - A container's pane title is set once, not twice; a redundant length check in share links and a no-op assignment in linkhandler.Launch are gone. - Tests call ansi.Strip and lastLines directly; gofmt -s on links.go. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
keb added 1 commit 2026-10-15 10:38:57 +02:00
refactor: drop dead code and use the standard library where it already does the job
CI / Checks (pull_request) Successful in 37s
Details
PR title / Conventional title (pull_request) Successful in 1s
Details
2d86af2c18 
No behaviour change.

- Deleted, unused or one-line wrappers: panes.RuneIndexes, Detail.Refresh (SetDescribed does it), SecretValue.Len, Query.Empty, Clipboard.Native, config.Load, get's value, Model.done, providerOf (cachefile.ProviderOf), contextWithTimeout, searchCoverage, startCrawl, showKeybindings, the expiry aliases and daysLeft.
- cmp.Or instead of the two cmpOr copies; slices.Sorted(maps.Keys(…)), maps.Copy, wg.Go, utf8.RuneCountInString and new(true) instead of loops and helpers that do the same.
- A container's pane title is set once, not twice; a redundant length check in share links and a no-op assignment in linkhandler.Launch are gone.
- Tests call ansi.Strip and lastLines directly; gofmt -s on links.go.
keb force-pushed refactor/dead-code-stdlib from 19299631b8 to 2d86af2c18 2026-10-15 10:38:57 +02:00 Compare
keb merged commit fae8e1cbb4 into main 2026-10-15 10:39:38 +02:00
keb deleted branch refactor/dead-code-stdlib 2026-10-15 10:39:38 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate-bot Bot keb (Karl-Einar Bergstrøm)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: keb/lazysecrets#76