Automatic detection of "Fixes" should be stricter
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript, node.js
- Domaine
- tooling
Piste de recherche
Utilisez PR #40613 et les métadonnées générées affichées dans cet issue comme cas de reproduction ; suivez le point d’entrée de node-core-utils qui détecte les métadonnées Fixes dans les messages de commit. Le travail est terminé lorsque la prose mentionnant « fix » ne produit plus de ligne Fixes incorrecte, tandis que les références Fixes valides continuent d’être détectées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In https://github.com/nodejs/node/pull/40613, [email protected] incorrectly adds a Fixes: metadata line:
----------------------------------- PR info ------------------------------------
Title crypto: fix build without scrypt (#40613)
Author Martin 'JaMa' Jansa <[email protected]> (@shr-project, first-time contributor)
Branch shr-project:jansa/scrypt -> nodejs:master
Labels crypto, c++, author ready
Commits 1
- crypto: fix build without scrypt
Committers 1
- Martin Jansa <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Voltrex <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Voltrex <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
--------------------------------------------------------------------------------
The commit message was:
crypto: fix build without scrypt
* add missing semicolon to fix:
In file included from ../src/node_crypto.h:47,
from ../src/node.cc:46:
../src/crypto/crypto_scrypt.h:80:2:
error: expected ';' after struct definition
80 | }
| ^
| ;
and fix typo in the comment
Signed-off-by: Martin Jansa <[email protected]>
(Original post in https://github.com/nodejs/node/pull/40613#issuecomment-966507251.)
- Langage dominant
- JavaScript
- Étoiles
- 313
- Forks
- 137
- Merge moyen
- 2 j 7 h
- PR mergées (30 j)
- 29
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.
Autres issues de nodejs/node-core-utils
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
nodejs/node-core-utils#664 · 1 commentaire ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
nodejs/node-core-utils#1202 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
nodejs/node-core-utils#1164 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
nodejs/node-core-utils#1160 · 3 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
nodejs/node-core-utils#1151 ·
Toutes les issues de nodejs/node-core-utils
Issues similaires
-
area/install-update comp/cli comp/desktop P3 sweeper:risk-compatibility type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
NousResearch/hermes-agent#122386 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
-
security
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
IBM/node-sdk-core#373 ·
-
docs web/
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100