Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

watch: a rebuild queued during a failed deploy is never deployed

オープン 初心者向け
  • 何か問題がありますか？
#48 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
78/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
node.js, typescript
領域
cli, devops

調査の方向性

packages/cli/src/watch.tsから始めて、triggerDeployがpromiseの成功時と失敗時の経路をどのように処理するかを追ってください。デプロイの実行中に変更を保存して失敗を再現し、その後、どちらの結果の場合もキューに入ったデプロイが実行されることを確認してください。また、#47で説明されているデプロイ保留によってブロックされないことを確認してください。

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

説明

Problem

In packages/cli/src/watch.ts, if a rebuild happens while a deploy is running, triggerDeploy sets deployQueued = true. The .then handler starts the queued deploy, but the .catch handler only resets isDeploying and never checks deployQueued.

So if the running deploy fails, a change saved during it is silently left undeployed until the next file change. That's often exactly the fix for the failure.

Repro

  1. In notation watch, start a deploy that will fail.
  2. While it's running, save a change.
  3. The failed deploy finishes, and the saved change is not deployed until another edit.

Fix

Drain the queue in both paths, e.g. move the deployQueued check into a .finally, or call the same "done" function from .then and .catch.

Note that while #47 is open, the queued deploy would then hang on the deployment hold, so the two are worth fixing together.

主要言語
TypeScript
スター
54
フォーク
1
平均マージ
1時間 45分
マージ済み PR（30日）
1

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

notationlabs/notation のほかの issue

notationlabs/notation の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

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

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