microsoft/Terminal

Make a second `LibraryIncludes.h` that includes WIL and TIL & have it at the bottom of every `precomp.h`

Open

#5182 aperta il 30 mar 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (3212 fork)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Meta

Metriche repository

Star
 (35.764 star)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

Can you file a follow-on task to perhaps make a second LibraryIncludes.h that includes WIL and TIL and have it at the bottom of every precomp.h?

I feel like you've done this to like 6 more projects as of this PR and I bet it will just keep coming.

We should probably put all super-powered headers at the bottom of every precomp.h.

Originally posted by @miniksa in https://github.com/microsoft/terminal/pull/5131

Guida contributor

Direzione di ricerca
Trova tutti i file precomp.h nel repository e aggiungi una direttiva #include per il nuovo LibraryIncludes.h in fondo a ciascuno. Il nuovo header deve includere gli header WIL e TIL.
Tech stack
cpp
Dominio
build system
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90