Se il tuo progetto tiene traccia degli articoli in sospeso nel codice sorgente con TODO
commenti, allora devi permetterlo.
Il fatto che la presenza di un TODO
commento nella richiesta pull stia disturbando dovrebbe dirti che tenere traccia degli elementi in sospeso nel codice sorgente è una cattiva idea. Le cose tendono a perdersi o essere ignorate in quel modo. Ora, se stai parlando di una richiesta pull a un "fork di lavoro", la situazione è diversa. Un "fork di lavoro" è proprio questo: un lavoro in corso. Ma una forcella del genere di solito non richiede una richiesta pull. Le "Regole della casa" qui suggerite sono per le richieste pull per il ramo principale .
Regola n. 1 della casa - Tutti i commit per il master devono essere pronti per il test, poiché il master viene creato ogni giorno dopo ogni check-in. Tali impegni dovrebbero includere anche eventuali test aggiuntivi richiesti.
Se la TODO
commento è presente perché il codice non è terminato o i test non sono finiti o il codice non è in alcun modo pronto per il test, quel codice appartiene a un commit locale, non a una richiesta pull. Chiamami quando è pronto.
Regola n. 2: tutte le informazioni relative ai problemi aperti vengono archiviate nel tracker dei problemi. Tutto. Note, scarabocchi, intuizioni, qualunque cosa.
Se la TODO
commento riguarda un problema aperto e non è una soluzione effettiva per quel problema, tali informazioni appartengono al tracker del problema. In questo modo, prima della chiusura di un problema, è possibile rivedere e verificare tutte le informazioni, se necessario, per assicurarsi che il problema sia effettivamente risolto.
Regola n. 3: tutte le informazioni relative alle attività di progetto in sospeso appartengono alla coda prioritaria (o qualunque sia il nome del sistema per quello).
Per chiarimenti, un'attività di progetto in sospeso è qualcosa che deve essere fatto nel progetto che ha una priorità prefissata, sia che si tratti di un difetto scoperto prima che generasse un ticket di emissione o dell'implementazione di un requisito di progettazione specifico, sia di uno dei componenti necessari di tale requisito.
Se la TODO
commento è 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 deve essere esaminato che è stato scoperto durante l'implementazione del nuovo codice, allora quelle informazioni appartengono alla coda prioritaria, sia su l'attività esistente o come nuova.
Regola n. 4 della casa - I suggerimenti appartengono alla Idea Box (o qualsiasi altra cosa).
Se la TODO
commento suggerisce un cambiamento nella progettazione o nell'implementazione del software, tali informazioni appartengono al riquadro delle idee del progetto, o "vNext", o "Design Notes", o qualsiasi altra cosa tu abbia per quel genere di cose.
Regola n. 5: i TODO
commenti non sono consentiti nel codice sorgente. PERIODO.
Se ti attieni a questa regola, non dovrai preoccuparti che nessuno segua nulla.