rewatch watcher: the lock-file exit path is duplicated
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 65/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- rust
- 領域
- build-system
調査の方向性
rewatch/src/watcher.rs の終了処理ブロックの 2 つのコピー（393 行目付近と 635 行目付近）から始めて、ロックファイルのチェックを行ごとに比較してください。共通のロジックを 1 つのヘルパー関数に移し、両方の呼び出し箇所がそれを使うようにしてください。動作は変えないでください。両方のパスがヘルパー関数を呼び出し、既存の rewatch の Rust テスト（rewatch/ で cargo test）が引き続き通れば完了です。
索引モデルが issue の本文から書いたものです。
説明
Found by differential semantic drift analysis from ece8b148 to a7721303.
What is broken
In rewatch/src/watcher.rs, the block that removes the lock file and exits with "Exiting... (lockfile removed)" appears twice: at L393 and, added by #8730, at L635, each with its own check for the lock file. A future change to the exit behaviour has to be made in both places.
Possible cause (speculation only)
A guess: #8730 fixed the full-rebuild path by copying the existing exit block into it, rather than factoring it into a shared function.
- 主要言語
- OCaml
- スター
- 7.5k
- フォーク
- 484
- 平均マージ
- 22時間 10分
- マージ済み PR（30日）
- 70
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rescript-lang/rescript のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
rescript-lang/rescript#8763 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
rescript-lang/rescript#8761 ·
メンテナーはふだん 1 日以内に返信
-
Issues from differential semantic drift analysis (ece8b148 to a7721303)対応中かも @cknitt が今日担当しました。 オープン
rescript-lang/rescript#8768 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 半日 初心者へのやさしさ 55/100
rescript-lang/rescript#8767 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 半日 初心者へのやさしさ 72/100
rescript-lang/rescript#8766 ·
メンテナーはふだん 1 日以内に返信
rescript-lang/rescript の issue をすべて見る
似ている issue
-
0.kind: enhancement 9.needs: package (update)
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
mesonbuild/wrapdb#2961 ·
メンテナーはふだん 1 日以内に返信
-
package-update
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
oSoWoSo/vOid_Community_repOsitory#330 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
awaiting-response bug needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
wildcard/caro#1562 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信