nodejs / nodejs/node-core-utils
Automatic detection of "Fixes" should be stricter
未关闭
还没有人认领这个 Issue。
bug
- 主要语言
- JavaScript
- 星标
- 314
- 派生
- 137
- 平均合并
- 2 天 8 小时
- 30 天内合并 PR
- 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 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
使用 PR #40613 和此 issue 中显示的生成元数据作为复现案例;跟踪从提交消息中检测 Fixes 元数据的 node-core-utils 入口点。完成的标准是,提及“fix”的文字不再生成不正确的 Fixes 行,同时仍能检测到有效的 Fixes 引用。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript, node.js
- 领域
- tooling
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 42/100