project-chip/connectedhomeip

[CI/CD] Not all CI steps use the same warning levels when compiling

Open

#30.854 aperta il 6 dic 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (1659 fork)batch import
CI/CD improvementsgood first issuetools

Metriche repository

Star
 (6410 star)
Metriche merge PR
 (Merge medio 8g 5h) (375 PR mergiate in 30 g)

Descrizione

Looks like at least some of our CI / builders let through unused variables. We should turn this on everywhere. see https://github.com/project-chip/connectedhomeip/pull/30793/commits/d3752baacd322a78bcffae4d78fc2b0c64afe72f

Also...some let through code with missing semicolons somehow. I don't even know how that can happen, but see https://github.com/project-chip/connectedhomeip/pull/30793/commits/a14846a9aa48f5f8f8c06603a3ebd97e9c67b636

Guida contributor

Direzione di ricerca
Identifica quali passi CI utilizzano livelli di avviso inferiori e aggiorna i flag del compilatore per imporre Werror per variabili non utilizzate e punti e virgola mancanti. Esamina i commit di riferimento forniti per vedere le modifiche specifiche necessarie.
Tech stack
ccpp
Dominio
ci cdbuild system
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Chiara
Prerequisiti
CI/CDC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40