nodejs / nodejs/node-core-utils
Automatic detection of "Fixes" should be stricter
オープン
まだ誰も着手していません。
bug
- 主要言語
- JavaScript
- スター
- 314
- フォーク
- 137
- 平均マージ
- 2日 8時間
- マージ済み PR(30日)
- 24
説明
In https://github.com/nodejs/node/pull/40613, node-core-utils@1.30.0 incorrectly adds a Fixes: metadata line:
----------------------------------- PR info ------------------------------------
Title crypto: fix build without scrypt (#40613)
Author Martin 'JaMa' Jansa <Martin.Jansa@gmail.com> (@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 <martin.jansa@lge.com>
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40613
Fixes: https://github.com/In
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
--------------------------------------------------------------------------------
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 <martin.jansa@lge.com>
(Original post in https://github.com/nodejs/node/pull/40613#issuecomment-966507251.)
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
PR #40613 と、この issue に表示されている生成済みメタデータを再現ケースとして使用し、コミットメッセージから Fixes メタデータを検出する node-core-utils のエントリーポイントを追跡します。完了条件は、「fix」に言及する文章によって誤った Fixes 行が生成されなくなり、有効な Fixes 参照は引き続き検出されることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript, node.js
- 領域
- tooling
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100