envoyproxy/envoy

Sharing support for clang-tidy (and maybe more) with other repositories

Open

#5.885 aberto em 8 de fev. de 2019

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
help wantedtech debt

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

Sharing support for clang-tidy (and maybe more) with other repositories

Description: The envoy-perf repository now runs clang-tidy in CI [1]. In order to so, some scripts had to be copied over from Envoy, resulting in code duplication. This was needed because some minor changes were required. It would be nice to have unified support for things like clang-format, perhaps via a shared submodule.

Links: [1] Envoy-perf pull request introducing clang-tidy in CI

Guia do colaborador

Direção de pesquisa
Investigue os scripts clang tidy existentes no Envoy e envoy perf, identifique partes comuns e divergentes e proponha um submódulo ou estrutura de script unificada.
Pilha de tecnologia
cpp
Domain
ci cd
Tipo Issue
Manutençã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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70