`rescript/cli/bins` checks Node.js >=20.11.0 while the packages require >=22
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 82/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- javascript, nodejs
- Bereich
- cli
Rechercherichtung
Beginne mit der Versionskonstante und der Fehlermeldung in cli/common/bins.cjs (etwa Z. 9 und Z. 69) und vergleiche sie dann mit engines.node im Root sowie packages/@rescript/*/package.json und .nvmrc. Fertig ist es, wenn die Laufzeitprüfung und ihre Meldung beide Node 22 verlangen, sodass Node 20.11–21 mit einer klaren Fehlermeldung abgelehnt wird. Prüfe die Einstellung für die Node-Version in der CI, um sicherzustellen, dass nichts anderes noch 20.11 erwartet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Found by differential semantic drift analysis from ece8b148 to a7721303.
What is broken
cli/common/bins.cjs L9 checks for Node.js >=20.11.0 and reports "ReScript requires Node.js >=20.11.0" (L69), while every other statement of the requirement says 22:
engines.nodeis">=22"in the rootpackage.jsonand in all fivepackages/@rescript/*/package.json(#8695);.nvmrcis22;CONTRIBUTING.mdsays v22.x or newer; CI tests "oldest supported" withnode-version: 22.
So on Node 20.11–21 the runtime check passes and users get no clear message, although the version is unsupported.
Related: #8694 and #8695, both listed in #8712; this is how the two interact, not a repeat of either.
Possible cause (speculation only)
A guess: #8694 copied the version constant from the previous bins.js while #8695 raised the requirement in parallel, and neither PR saw the other's change.
- Vorherrschende Sprache
- ReScript
- Sterne
- 7.5k
- Forks
- 483
- Ø Merge
- 1 T. 1 Std.
- Gemergte PRs (30 T.)
- 120
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus rescript-lang/rescript
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
rescript-lang/rescript#8833 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
rewatch watch on Linux stops watching a source folder when an overlapping recursive source folder is removed from rescript.jsonEvtl. vergeben @cknitt hat das vor 3 Tagen übernommen. Offen
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8825 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8806 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
rescript-lang/rescript#8798 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 Ein halber Tag Anfängerfreundlichkeit 58/100
rescript-lang/rescript#8757 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in rescript-lang/rescript
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
jbacalso24/mimry#291 ·
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
eli-alvarado/us-watchlist-alerts#11 ·
Maintainer antworten meist innerhalb von 1 Tag
-
🐛 Bug: `--clear-screen` not very useful (at least not for dealing with large test/error output)Offenstatus: in triage type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
mochajs/mocha#6428 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:cli bug good first issue priority:medium
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
rtk-ai/rtk#4589 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Change oauth client name?Evtl. vergeben @nealrichardson hat das heute übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 66/100
posit-dev/rsconnect-python#870 ·
Maintainer antworten meist innerhalb von 1 Tag