nodejs / nodejs/node-core-utils
Erroneous "Commit message is missing PR-URL" error
未关闭
还没有人认领这个 Issue。
bug
- 主要语言
- JavaScript
- 星标
- 313
- 派生
- 137
- 平均合并
- 2 天 8 小时
- 30 天内合并 PR
- 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 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 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