Automatic detection of "Fixes" should be stricter

オープン

#592 2021年11月11日に作成コメント 1 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 42/100

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 停滞
技術スタック: javascript, node.js
領域: tooling

調査の方向性

PR #40613 と、この issue に表示されている生成済みメタデータを再現ケースとして使用し、コミットメッセージから Fixes メタデータを検出する node-core-utils のエントリーポイントを追跡します。完了条件は、「fix」に言及する文章によって誤った Fixes 行が生成されなくなり、有効な Fixes 参照は引き続き検出されることです。

索引モデルが issue の本文から書いたものです。

説明

bug

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.)}