Open
Description
In nodejs/node#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 nodejs/node#40613 (comment).)
Activity
richardlau commentedon Nov 11, 2021
I think it's being picked out of the body of the pull request description. The same issues is also often seen with the npm update pull requests.