readthedocs/readthedocs.org

Use a named tuple for tags and branches data

Open

#7798 aperta il 5 gen 2021

Vedi su GitHub
 (5 commenti) (1 reazione) (0 assegnatari)Python (3748 fork)batch import
AcceptedGood First IssueImprovement

Metriche repository

Star
 (7765 star)
Metriche merge PR
 (Merge medio 10g 6h) (44 PR mergiate in 30 g)

Descrizione

We are passing a list of dictionaries with verbose_name and identifier, we should use a data structure for this instead to avoid bugs. We need to make sure that celery can serialize named tuples.

ref https://github.com/readthedocs/readthedocs.org/pull/7548#discussion_r542636920

Guida contributor

Direzione di ricerca
Esplora come sostituire la lista di dizionari con una tupla nominata e verifica che Celery possa serializzarla. Consulta la discussione PR collegata per il contesto.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65