nodejs / nodejs/node-core-utils
setting up remote-upstream through ncu
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 314
- Forks
- 137
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 24
Beschreibung
I am on a new laptop and forgot some configs in my project specific config and failed the cq due to column length > 72 characters, therefore decided to land the PR manual by using core-util.
Followed the steps to:
pre: installed @node-core/utils through npm
- create a scoped token
- put token into
~/.ncurc - run
git node land 57732 - followed the prompt to setup remote-url and branch main to main
➜ node git:(add-lower-bound-range-test) ✗ ncu-config list
branch: main
upstream: remote-upstream
and then I encountered the following error message:
at runSync (file:///Users/jake/.nvm/versions/node/v23.11.0/lib/node_modules/@node-core/utils/lib/run.js:109:11)
at new Session (file:///Users/jake/.nvm/versions/node/v23.11.0/lib/node_modules/@node-core/utils/lib/session.js:30:28)
at new LandingSession (file:///Users/jake/.nvm/versions/node/v23.11.0/lib/node_modules/@node-core/utils/lib/landing_session.js:24:5)
at main (file:///Users/jake/.nvm/versions/node/v23.11.0/lib/node_modules/@node-core/utils/components/git/land.js:171:17)
I think it is trying to find remote-upstream as after I did git remote add remote-upstream https://github.com/nodejs/node.git the issue was resolved. However, this confused me for a few minutes, shouldn't it default to just upstream or attempt to add remote-upstream for the user automatically?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere den Fehler mit der gemeldeten ncu-config-Liste und dem git node land-Ablauf. Beginne dann in components/git/land.js und lib/landing_session.js, wobei der Command-Runner in lib/run.js liegt. Prüfe, wie der konfigurierte remote-upstream aufgelöst wird, wenn er fehlt, und definiere als erledigt, dass dieser fehlende Remote konsistent mit dem vorgesehenen Upstream-Verhalten behandelt und das Ergebnis durch einen Test abgedeckt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- git, javascript, nodejs
- Bereich
- cli, developer-experience, devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100