project-chip/connectedhomeip

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

Open

#30 854 ouverte le 6 déc. 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (1 659 forks)batch import
CI/CD improvementsgood first issuetools

Métriques du dépôt

Stars
 (6 410 stars)
Métriques de merge PR
 (Merge moyen 8j 5h) (375 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Identifiez quelles étapes CI utilisent des niveaux d'avertissement inférieurs et mettez à jour les indicateurs du compilateur pour appliquer Werror pour les variables inutilisées et les points virgules manquants. Examinez les références de commit fournies pour voir les modifications spécifiques nécessaires.
Stack technique
ccpp
Domaine
ci cdbuild system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CI/CDC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40