angular / angular/dev-infra

[release tool]: run nvm gracefully to ensure caretaker uses correct node version for publish branch

オープン
#312 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
domain: release automation
主要言語
JavaScript
スター
77
フォーク
84
平均マージ
2日 5時間
マージ済み PR(30日)
37

説明

The merge tool will currently check out other branches, such as patch, LTS etc. For that the node versions could be different and breaking. This currently results in some inconvenient Yarn install engines error mid-publishing in a way where recovering is rather cumbersome..

We should run `nvm` automatically, gracefully as not everbody might have it. If it fails, we print a warning and proceed running the Yarn install. That one could still fail in such cases, but this is expected then.

Note: We have considered using the Bazel-managed Node version (that would be ideal).. but we cannot let Bazel install the node modules as Bazel itself is actually made available through the `@bazel/bazelisk` package (caretaker might not have the proper Bazel version globally installed; if at all)

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

merge tool の branch-checkout と Yarn install のフローから着手し、特に publishing が patch または LTS ブランチに切り替わる箇所を確認してください。現在ツールが環境セットアップをどのように処理しているかを確認し、そのうえで nvm の試行が適切に処理され、利用できない場合や失敗した場合に警告が出力され、Yarn install が引き続き実行されることを検証してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
release, tooling
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
48/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。