nodejs / nodejs/core-validate-commit

Invalid Commit Message

Aberta
#117 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Linguagem predominante
JavaScript
Estrelas
55
Forks
62
Merge médio
4d 9h
PRs com merge (30d)
1

Descrição

This PR was merged with a non-imperative verb starting the commit message: benchmark: added new benchmarks for blob.

From what the docs says, it should be a invalid message.

The CI did raised an error about the length (that I fix) but didn't raise an error about this verb and it was merged.

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece pelas diretrizes de mensagens de commit e pela validação exercida por PR 49730; em seguida, rastreie a verificação que relatou o erro de comprimento da mensagem. Verifique como o verbo inicial do assunto do commit é avaliado. Considera-se concluído quando um assunto não imperativo, como “added new benchmarks for blob”, é rejeitado pelo CI, enquanto uma formulação imperativa válida é aprovada.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript
Domínio
ci-cd, testing-qa
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.