Automatic detection of "Fixes" should be stricter

Đang mở

#592 mở ngày 11/11/2021 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó: 3/5
Thời gian dự kiến: 1-2 ngày
Mức phù hợp với người mới: 42/100

Loại issue: Lỗi
Độ rõ ràng: Khá rõ ràng
Mức độ hoạt động: Đình trệ
Công nghệ: javascript, node.js
Lĩnh vực: tooling

Hướng nghiên cứu

Sử dụng PR #40613 và siêu dữ liệu được tạo hiển thị trong issue này làm trường hợp tái hiện; lần theo entry point của node-core-utils để phát hiện siêu dữ liệu Fixes từ các thông điệp commit. Hoàn thành có nghĩa là phần văn bản đề cập đến “fix” không còn tạo ra một dòng Fixes không chính xác, trong khi các tham chiếu Fixes hợp lệ vẫn tiếp tục được phát hiện.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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