Commit a1603c893b

a1603c893b54936d4a94b452452484f7c5c49171

parent: 33f33afe31

Verified · cmc

cmc <hello@cleberg.net> · 2026-09-22 21:36 UTC

mr: say why labels load outside load()

The comment restated the rule (read in the Labels section, not in
load()) without saying why. Labels colours are secondary and the
detail screen's tests count load()'s four reads.

Layout: unified · split

gitbay/MRs/MRDetailViewModel.swift +2 −2
@@ -85,8 +85,8 @@ final class MRDetailViewModel {
85 } 85 }
86 86
87 /// The repository's labels, for chip colours. Read on first use from 87 /// The repository's labels, for chip colours. Read on first use from
88 /// the Labels section rather than in `load()`, which stays at four 88 /// the Labels section, not in `load()`: colours are secondary, and the
89 /// reads. 89 /// detail screen's tests count `load()`'s four reads.
90 private(set) var colors = LabelColors() 90 private(set) var colors = LabelColors()
91 private var labelsLoaded = false 91 private var labelsLoaded = false
92 92