daphne-project/daphne

Formatting in tests

Aberta

#849 aberto em 12 de out. de 2024

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C++ (84 forks)auto 404
Testsgood first issue

Métricas do repositório

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

Description

Since we added the .clang-format file, some of the tests that were formatted look a bit off now. For example, the brace initialized std::vector here are no longer formatted as a 4x6 matrix.

One possible solution is to check out the clang-format settings and figure out if there's a setting that works for our test code.

https://clang.llvm.org/docs/ClangFormatStyleOptions.html

Another one would be to put // clang-format [off | on] around such sections we do not want to have formatted.

Guia do colaborador

Direção de pesquisa
Revise a configuração atual do .clang format e teste o efeito de diferentes opções no arquivo de teste específico. Alternativamente, proponha adicionar comentários clang format off/on ao redor das inicializações da matriz.
Pilha de tecnologia
cpp
Domain
developer experience
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.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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.
80