envoyproxy/envoy

Support some extensions from C++20 in Windows CI

Open

#18 489 ouverte le 6 oct. 2021

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/buildarea/windowsenhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Google style permits designated initializers from C++20. MSVC is strict about C++17. The request is to allow Google style code in Windows CI even if it's not fully compliant with C++17. An option is to use C++20 in MSVC.

Guide contributeur

Direction de recherche
Le problème demande d'autoriser les initialiseurs désignés dans le CI Windows en passant MSVC à C++20. Actuellement, le CI impose C++17. Pour implémenter, modifiez la configuration du CI Windows (probablement dans .bazelrc ou les scripts CI) pour ajouter /std:c++20 pour les builds MSVC. Assurez vous que les autres parties de la base de code restent compatibles et que le changement ne casse pas d'autres vérifications. Consultez le guide de style Google pour confirmer que les initialiseurs désignés sont autorisés.
Stack technique
ccpp
Domaine
backendci cd
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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
C++ build configurationWindows CI
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80