tech-debt: Fix abused repo status fields (broken, is_empty) #317

New Issue

2026-05-31T14:20:13Z

jmiller commented

2026-05-31 14:20:13 +00:00

Summary

Repo status fields are being used incorrectly.

Locations

models/repo/repo.go:318 - the status broken and is_empty were abused
models/repo/repo.go:368 - HTMLURL used in mail templates without ctx
models/repo/repo_list.go:47 - remove in favor of maps.values

Action Required

Clarify semantics of repo status fields and fix the HTMLURL context issue for mail templates.

Created by Claude Opus 4.6

## Summary Repo status fields are being used incorrectly. ## Locations - `models/repo/repo.go:318` - the status broken and is_empty were abused - `models/repo/repo.go:368` - HTMLURL used in mail templates without ctx - `models/repo/repo_list.go:47` - remove in favor of maps.values ## Action Required Clarify semantics of repo status fields and fix the HTMLURL context issue for mail templates. --- *Created by Claude Opus 4.6*

jmiller added the tech-debt upstream labels 2026-05-31 14:20:13 +00:00

jmiller referenced this issue from a commit

2026-05-31 16:04:36 +00:00

fix: resolve tech-debt batch 3 — remove deprecated functions, use stdlib

jmiller referenced this issue

2026-05-31 16:05:02 +00:00

fix: tech-debt batch 3 — remove deprecated functions, use stdlib #357

Sign in to join this conversation.