nodejs / nodejs/node-core-utils
Erroneous "Commit message is missing PR-URL" error
オープン
まだ誰も着手していません。
bug
- 主要言語
- JavaScript
- スター
- 314
- フォーク
- 137
- 平均マージ
- 2日 8時間
- マージ済み PR(30日)
- 24
説明
Seen here: https://github.com/nodejs/node/pull/35405
When i proceed anyway, each message nevertheless has all requisite info:
node on git:v12.x-staging ❯ git rebase upstream/v12.x-staging -i -x "git node land --amend"
Executing: git node land --amend
⚠ Found PR-URL: , skipping..
--------------------------------- New Message ----------------------------------
module: named exports for CJS via static analysis
PR-URL: https://github.com/nodejs/node/pull/35249
Backport-PR-URL: https://github.com/nodejs/node/pull/35405
Reviewed-By: Mary Marchini <oss@mmarchini.me>
Reviewed-By: Geoffrey Booth <webmaster@geoffreybooth.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
--------------------------------------------------------------------------------
? Use this message? Yes
[detached HEAD 65ffce152b] module: named exports for CJS via static analysis
Author: Guy Bedford <guybedford@gmail.com>
Date: Thu May 14 22:40:37 2020 -0700
20 files changed, 1926 insertions(+), 85 deletions(-)
create mode 100755 deps/cjs-module-lexer/.gitignore
create mode 100755 deps/cjs-module-lexer/LICENSE
create mode 100755 deps/cjs-module-lexer/README.md
create mode 100755 deps/cjs-module-lexer/lexer.js
create mode 100755 deps/cjs-module-lexer/package.json
create mode 100644 test/es-module/test-esm-cjs-exports.js
create mode 100644 test/fixtures/es-modules/cjs-exports.mjs
create mode 100644 test/fixtures/es-modules/exports-cases.js
create mode 100644 test/fixtures/es-modules/exports-cases2.js
create mode 100644 test/fixtures/es-modules/exports-cases3.js
Executing: git node land --amend
⚠ Found PR-URL: , skipping..
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、issue に示されている rebase 中に報告された git node land --amend コマンドを再現し、次にコミットメッセージから PR-URL を読み取る仕組みを追跡します。issue にはファイルやテストのパスが示されていないため、関連する CLI エントリーポイントを特定し、有効な PR-URL を含むメッセージのカバレッジを追加します。コマンドが PR-URL を欠落または空として報告しなくなれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- cli
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100