daphne-project/daphne

Formatting in tests

Aperta

#849 aperta il 12 ott 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (84 fork)auto 404
Testsgood first issue

Metriche repository

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

Descrizione

Since we added the .clang-format file, some of the tests that were formatted look a bit off now. For example, the brace initialized std::vector here are no longer formatted as a 4x6 matrix.

One possible solution is to check out the clang-format settings and figure out if there's a setting that works for our test code.

https://clang.llvm.org/docs/ClangFormatStyleOptions.html

Another one would be to put // clang-format [off | on] around such sections we do not want to have formatted.

Guida contributor

Direzione di ricerca
Esamina la configurazione corrente di .clang format e testa l'effetto di diverse opzioni sul file di test specifico. In alternativa, proponi di aggiungere commenti clang format off/on intorno alle inizializzazioni della matrice.
Tech stack
cpp
Dominio
developer experience
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80