nodejs / nodejs/node-core-utils

Erroneous "Commit message is missing PR-URL" error

Ouverte
#506 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug
Langage dominant
JavaScript
Étoiles
313
Forks
137
Merge moyen
2 j 8 h
PR mergées (30 j)
24

Description

Seen here: https://github.com/nodejs/node/pull/35405

When i proceed anyway, each message nevertheless has all requisite info:

node on git:v12.x-staging ❯ git rebase upstream/v12.x-staging -i -x "git node land --amend"
Executing: git node land --amend
   ⚠  Found PR-URL: , skipping..
--------------------------------- New Message ----------------------------------
module: named exports for CJS via static analysis

PR-URL: https://github.com/nodejs/node/pull/35249
Backport-PR-URL: https://github.com/nodejs/node/pull/35405
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Geoffrey Booth <webmaster@geoffreybooth.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
--------------------------------------------------------------------------------
? Use this message? Yes
[detached HEAD 65ffce152b] module: named exports for CJS via static analysis
 Author: Guy Bedford <guybedford@gmail.com>
 Date: Thu May 14 22:40:37 2020 -0700
 20 files changed, 1926 insertions(+), 85 deletions(-)
 create mode 100755 deps/cjs-module-lexer/.gitignore
 create mode 100755 deps/cjs-module-lexer/LICENSE
 create mode 100755 deps/cjs-module-lexer/README.md
 create mode 100755 deps/cjs-module-lexer/lexer.js
 create mode 100755 deps/cjs-module-lexer/package.json
 create mode 100644 test/es-module/test-esm-cjs-exports.js
 create mode 100644 test/fixtures/es-modules/cjs-exports.mjs
 create mode 100644 test/fixtures/es-modules/exports-cases.js
 create mode 100644 test/fixtures/es-modules/exports-cases2.js
 create mode 100644 test/fixtures/es-modules/exports-cases3.js
Executing: git node land --amend
   ⚠  Found PR-URL: , skipping..

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 reproduire la commande signalée git node land --amend pendant le rebase montré dans l’issue, puis suivez la façon dont elle lit la PR-URL dans le message du commit. L’issue ne fournit aucun chemin de fichier ou de test ; localisez donc le point d’entrée CLI pertinent et ajoutez une couverture pour un message contenant une PR-URL valide. C’est terminé lorsque la commande ne signale plus que la PR-URL est absente ou vide.

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

Évaluation

Stack technique
javascript
Domaine
cli
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

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