microsoft/Terminal

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

Open

#5.182 aberto em 30 de mar. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Meta

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Localize todos os arquivos precomp.h no repositório e adicione uma diretiva #include para o novo LibraryIncludes.h no final de cada um. O novo cabeçalho deve incluir os cabeçalhos WIL e TIL.
Pilha de tecnologia
cpp
Domain
build system
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90