Se il tuo progetto tiene traccia degli elementi in sospeso nel codice sorgente con i commenti di TODO
, devi permetterlo.
Il fatto che la presenza di un commento di TODO
nella richiesta pull sia un bug, dovresti dirti che tenere traccia degli elementi in sospeso nel codice sorgente è una cattiva idea. Le cose tendono a perdersi o ignorate in questo modo. Ora, se stai parlando di una richiesta di pull a un "fork di lavoro", la situazione è diversa. Un "fork di lavoro" è proprio questo: un work in progress. Ma una forcella come quella di solito non richiede una richiesta di pull. Le "Regole della casa" qui suggerite sono per le richieste di pull per il ramo master .
House Rule # 1 - Tutti i commit su master dovrebbero essere pronti per essere testati, dato che master viene costruito ogni giorno dopo ogni check-in. Tali commit dovrebbero includere anche eventuali test aggiuntivi richiesti.
Se il commento TODO
è presente perché il codice non è finito, oi test non sono terminati, o il codice non è in alcun modo pronto per il test, allora quel codice appartiene a un commit locale, non a un richiesta di pull. Chiamami quando è pronto.
Regola n. 2: tutte le informazioni relative ai problemi aperti sono memorizzate nel tracker dei problemi. Tutto. Note, scarabocchi, intuizioni, qualsiasi cosa.
Se il commento TODO
riguarda un problema aperto e non è una correzione effettiva per quel problema, allora tali informazioni appartengono al tracker dei problemi. In questo modo, prima che un problema venga chiuso, tutte le informazioni possono essere riviste e verificate, se necessario, per assicurarti che il problema sia effettivamente risolto.
Regola n. 3: tutte le informazioni relative alle attività in sospeso appartengono alla coda di priorità (o qualunque sia il nome del sistema).
Per chiarimenti, un'attività in progetto in sospeso è qualcosa che deve essere fatto nel progetto che ha una priorità prestabilita, che si tratti di un difetto scoperto prima della generazione di un ticket di emissione o dell'implementazione di un requisito di progettazione specifico, o uno dei componenti necessari di tale requisito.
Se il commento di TODO
è lì per dire che il nuovo codice avrà un impatto su un'attività in sospeso, o per indicare qualcos'altro nella base di codice che ha bisogno di essere guardato durante l'implementazione del nuovo codice, allora quella informazione appartiene a la coda di priorità, sull'attività esistente o come nuova.
Regola n. 4: i suggerimenti appartengono all'Idea Box (o qualsiasi altra cosa).
Se il commento di TODO
suggerisce un cambiamento nella progettazione o nell'implementazione del software, allora quell'informazione appartiene alla casella idea del progetto, o "vNext", o "Design Notes", o qualsiasi altra cosa tu abbia per quel tipo di cosa.
La regola di casa n. 5 - i commenti di TODO
non sono consentiti nel codice sorgente. PERIODO.
Se ti attieni a questa regola, non dovrai preoccuparti di nessuno che risponda a qualcosa.