envoyproxy/envoy

Support some extensions from C++20 in Windows CI

Open

#18.489 aperta il 6 ott 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/buildarea/windowsenhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema richiede di consentire gli inizializzatori designati nella CI di Windows passando MSVC a C++20. Attualmente, la CI impone C++17. Per implementare, modificare la configurazione della CI di Windows (probabilmente in .bazelrc o negli script CI) per aggiungere /std:c++20 per le build MSVC. Assicurarsi che le altre parti della codebase rimangano compatibili e che la modifica non interrompa altri controlli. Rivedere la guida di stile Google per confermare che gli inizializzatori designati sono consentiti.
Tech stack
ccpp
Dominio
backendci cd
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
C++ build configurationWindows CI
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80