nodejs / nodejs/node-core-utils

git node land fails with mismatched commits

Offen
#904 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
314
Forks
137
Ø Merge
2 T. 8 Std.
Gemergte PRs (30 T.)
24

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit dem Befehl git node land und reproduziere das gemeldete Verhalten anhand von PR 54990 und den abgerufenen Commits. Verfolge, warum der Befehl einen unerwarteten Commit und einen fehlenden Commit meldet; die Aufgabe ist abgeschlossen, wenn die Abweichung korrekt behandelt und das Ergebnis des Befehls anhand der PR-Daten verifiziert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
git, github, javascript
Bereich
cli, devtools
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.