sourcegraph/sourcegraph-public-snapshot

insights: how to represent language stats time scope

Aperta

#27.151 aperta il 5 nov 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Go (1374 fork)auto 404
backendcode-insightsdebtgood first issueinsights-settings-migration

Metriche repository

Star
 (10.290 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I guess my only concern here is that the semantic of a 0 value interval is ambiguous. What does that mean to a client, and how should one execute the search queries to generate this series based on this information?

I wonder if we just should have a new TimeScope type for a single time point (the most recent for example)? The intention behind the TimeScope was to ultimately have multiple types, some of which would have an interval and others would be different. Maybe this is a justification here? What do you think?

Originally posted by @coury-clark in https://github.com/sourcegraph/sourcegraph/pull/27102#discussion_r743760401

Guida contributor

Direzione di ricerca
Esamina la PR #27102 collegata e la discussione per capire l'implementazione corrente di TimeScope. Quindi proponi una nuova variante di TimeScope per un singolo punto temporale, considerando come i client interpreterebbero un valore di intervallo zero.
Tech stack
go
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50