Duzzuti/passwd-manager

[NTH] Ensure code quality

Aberta

#18 aberto em 1 de jun. de 2023

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

Métricas do repositório

Stars
 (9 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

We should ensure that functions should have two properties:

  1. The name is relevant for what they do
  2. The function does one single thing.

Specially if we notice that property number 2 isn't fulfilled we should split the function into as many different things is doing. Following this idea makes the code much easier to read and debug.

  • Difficulty (1-10): 2

Guia do colaborador

Direção de pesquisa
Revise cada função no código, verifique se os nomes correspondem ao comportamento e se as funções fazem uma única coisa. Caso contrário, renomeie ou divida.
Pilha de tecnologia
c
Domain
backend
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75