nodejs / nodejs/node-core-utils

git node land fails with mismatched commits

未关闭
#904 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

主要语言
JavaScript
星标
314
派生
137
平均合并
2 天 8 小时
30 天内合并 PR
24

描述

Landing 54990 fails with a new error that I've never seen before.

❯ git node land 54990
✔  Done loading data for nodejs/node/pull/54990
----------------------------------- PR info ------------------------------------
Title      src: replace splitstring with built-in (#54990)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     anonrig:replace-stringviewsplit -> nodejs:main
Labels     c++, lib / src, needs-ci, commit-queue-failed, dont-land-on-v18.x, dont-land-on-v20.x
Commits    1
 - src: replace splitstring with built-in
Committers 1
 - Yagiz Nizipli <yagiz@cloudflare.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/54990
Reviewed-By: Daniel Lemire <daniel@lemire.me>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Wed, 18 Sep 2024 01:10:06 GMT
   ✔  Approvals: 5
   ✔  - Daniel Lemire (@lemire): https://github.com/nodejs/node/pull/54990#pullrequestreview-2312621536
   ✔  - Chengzhong Wu (@legendecas) (TSC): https://github.com/nodejs/node/pull/54990#pullrequestreview-2317889526
   ✔  - Robert Nagy (@ronag) (TSC): https://github.com/nodejs/node/pull/54990#pullrequestreview-2317895154
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/54990#pullrequestreview-2319866121
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/54990#pullrequestreview-2572079210
   ✘  Last GitHub CI failed
   ℹ  Last Full PR CI on 2025-02-09T19:27:37Z: https://ci.nodejs.org/job/node-test-pull-request/65094/
⠋ Querying data for job/node-test-pull-request/65094/   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ This PR is not ready to land, do you want to continue? Yes
⠙ Querying API for job/node-test-pull-request/65094/   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
✔ Do you want to try reset the local main branch to upstream/main? No
⠼ Downloading patch for 54990From https://github.com/nodejs/node
 * branch                  refs/pull/54990/merge -> FETCH_HEAD
✔  Fetched commits as 03e9adf876e2..6511d48523cb
--------------------------------------------------------------------------------
   ✘  Mismatched commits:
Unexpected commit 6511d48523cb02b9323a810ff0ea25d55a9665ac
Missing commit d9790fef541096f4c7a8d869a21440e318117964

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

git node land 命令开始,使用 PR 54990 及其获取的提交复现报告中的行为。追踪该命令为何报告了一个意外提交和一个缺失提交;当不匹配得到正确处理,并且命令的结果已根据 PR 数据完成验证时,即视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
git, github, javascript
领域
cli, devtools
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
需要澄清
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。