project-chip/connectedhomeip

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

Open

#30.854 geöffnet am 6. Dez. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (1.659 Forks)batch import
CI/CD improvementsgood first issuetools

Repository-Metriken

Stars
 (6.410 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T 5h) (375 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Identifiziere, welche CI Schritte niedrigere Warnstufen verwenden, und aktualisiere die Compiler Flags, um Werror für ungenutzte Variablen und fehlende Semikolons zu erzwingen. Sieh dir die bereitgestellten Commit Referenzen an, um die erforderlichen spezifischen Änderungen zu sehen.
Tech Stack
ccpp
Domain
ci cdbuild system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
CI/CDC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40