nodejs / nodejs/core-validate-commit
Invalid Commit Message
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 55
- Forks
- 62
- Merge moyen
- 4 j 9 h
- PR mergées (30 j)
- 1
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par les directives relatives aux messages de commit et la validation exercée par PR 49730, puis suivez le contrôle qui a signalé l’erreur de longueur du message. Vérifiez comment le verbe initial du sujet du commit est évalué. C’est terminé lorsqu’un sujet non impératif tel que «added new benchmarks for blob» est rejeté par CI, tandis qu’une formulation impérative valide passe.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- ci-cd, testing-qa
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100