diff options
Diffstat (limited to 'config')
| -rw-r--r-- | config/detekt/detekt.yml | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/config/detekt/detekt.yml b/config/detekt/detekt.yml index cbb4390..8e94ed5 100644 --- a/config/detekt/detekt.yml +++ b/config/detekt/detekt.yml @@ -20,14 +20,17 @@ style: complexity: TooManyFunctions: - thresholdInInterfaces: 21 # Room DAOs are broad by nature -- each query is a method, and + thresholdInInterfaces: 22 # Room DAOs are broad by nature -- each query is a method, and # splitting one table's access across several interfaces to satisfy a count would make the # data layer harder to follow, not easier. Raised from 20 when EpisodeDao picked up # downloadedFromServer (the skip-refresh filter): one more genuinely distinct query, not - # scope creep. - thresholdInClasses: 20 # same for the view model: it is a flat list of one-line user actions + # scope creep. Raised again to 22 for waitingForServer, which cannot reuse queuedDownloads: + # a bounded retry must go newest first, or episodes the server has aged out of its newest-8 + # window would starve the ones it still has. + thresholdInClasses: 21 # same for the view model: it is a flat list of one-line user actions # (enqueue, download, mark played), and splitting it by count rather than by meaning would - # scatter related behaviour across types for no gain. + # scatter related behaviour across types for no gain. 21 with downloadFromPublisher, the + # explicit choice the episode screen offers a waiting download. naming: FunctionNaming: |