nodejs / nodejs/core-validate-commit

signed-off-by: skip rule when PR-URL trailer is present not only Backport-PR-URL?

Ouverte Adaptée aux débutants
#149 2 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

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

The signed-off-by rule currently skips backports identified by a Backport-PR-URL: trailer. This works for already-landed backport commits, but fails for backport PRs at review time because the commit-queue only adds Backport-PR-URL: at land time. even though the contributing doc explicitly exempts backports.

Rule 1 says:
https://github.com/nodejs/node/blob/e15f90559bc62b1e9a8296f1fc8654f5be261baa/doc/contributing/pull-requests.md?plain=1#L207-L208

Rule 2 says:
https://github.com/nodejs/node/blob/e15f90559bc62b1e9a8296f1fc8654f5be261baa/doc/contributing/backporting-to-release-lines.md?plain=1#L77

The lint (in core-validate-commit) says: if no Signed-off-by: AND no Backport-PR-URL: (which is added by the commit-queue), fail.

Suggestion

Treat PR-URL: trailer as the same kind of skip signal as Backport-PR-URL:. Both trailers are added by the commit-queue at land time they're never contributor-authored. Their presence on a commit being validated is a strong heuristic that the commit was either cherry-picked from a previously-landed PR (the dominant case, all backports), or is itself a previously-landed commit having its message re-validated.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la règle Signed-off-by dans core-validate-commit et suivez son traitement actuel du trailer Backport-PR-URL:. Vérifiez les tests de validation, s’ils existent, et confirmez que les commits avec PR-URL: sont acceptés sans Signed-off-by, tandis que les autres commits conservent l’exigence actuelle.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
tooling
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
68/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.