microsoft/Terminal

Re-enable C26814 for constexpr locals

Aberta

#2.941 aberto em 27 de set. de 2019

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (1 responsável)C++ (9.275 forks)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Meta

Métricas do repositório

Stars
 (103.173 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

This showed up in a recent compiler edition and we aren't ready for all the build breaks it is causing.

Many of the issues look fixable, but some of them look upstream in wil (though I suspect it when it says that an HR can be figured out at compile time...)

Guia do colaborador

Direção de pesquisa
Investigue o aviso do compilador C26814 para variáveis locais constexpr no codebase do Terminal. Identifique todos os locais ofensores e determine se eles são corrigíveis ou se são necessárias alterações em dependências upstream como wil. Prepare um plano para reabilitar o aviso após corrigir os problemas.
Pilha de tecnologia
cpp
Domain
desktop
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Windows development
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30