Fix test fixture, README provenance and remaining Windows-only polish
まだ誰も着手していません。
評価
調査の方向性
plugin/scripts/godebugdiff_test.go の TestCompareWindowsOnly フィクスチャから始めます。ここでは winreadlinkvolume が Changed: 26 と期待されていますが、本来は 23 であるべきです。compare() 内の c.WindowsOnly = isWindowsOnly(k) をコメントアウトして項目 1 を再現し、失敗メッセージの URL を確認してください。フィクスチャが #go-123 の URL を伴う 23 を検証し、Before、After、Package を確認しており、examples/README.md がファイルごとのツールチェーンを一箇所にまとめて記載していれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Follow-up from #12:
- TestCompareWindowsOnly (plugin/scripts/godebugdiff_test.go) gives winreadlinkvolume Changed: 26. The real value is 23, so the change gets the URL go.dev/doc/godebug#go-126 instead of #go-123. To reproduce: comment out c.WindowsOnly = isWindowsOnly(k) in compare() and run the test; the failure message prints URL:https://go.dev/doc/godebug#go-126. Use 23. The test also ignores the second value compare() returns and never checks Before, After or Package. Asserting them costs nothing.
- The WindowsOnly field comment now says "Windows-specific setting", but isWindowsOnly only checks the "win" name prefix. Make the comment match the rule, or derive the mark from an explicit list.
- examples/README.md line 12 still says the real outputs were made with Go 1.27.3, and the new line 24 says cobra.txt was redone with Go 1.27.4. Put the toolchain for each file in one place, for example a column in the table.
- Item 3 of #11 is still open, although #12 closes #11: the "No binary imports the package, so no effect" section still adds "(Windows only)". To reproduce: run from 1.22 to 1.23 on a module whose only binary is an empty func main() {}. Also, windowsOnly still uses omitempty while goCommand is always emitted, and the report does not tell Linux-only services that they can ignore the pins for Windows-only settings.
GillMacedo
- 主要言語
- Go
- スター
- 0
- フォーク
- 3
- 平均マージ
- 4時間 55分
- マージ済み PR（30日）
- 4
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
GillMacedo/godebugdiff のほかの issue
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
Add a --only flag to filter by package対応中かも @snehaapratap が 13 日前に担当しました。 オープンgood first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
-
Read the go line from go.work in workspace mode対応中かも @ducnhan123-co が 12 日前に担当しました。 オープンgood first issue
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
GillMacedo/godebugdiff#4 · コメント 1 件 ·
-
good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
GillMacedo/godebugdiff の issue をすべて見る
似ている issue
-
good first issue severity: low tech-debt
難易度 2/5 半日 初心者へのやさしさ 75/100
TMHSDigital/subenum#329 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
GillMacedo/racesift#14 ·
-
IsTextFile reads the whole file into memory (OOM on large files in the file watcher)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
gtsteffaniak/filebrowser#3395 ·
メンテナーはふだん 1 日以内に返信
-
effort:s impact:low raised-by:worker risk:low
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
medici-finance/assay#6496 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
kind/bug require/reproduce require/severity
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信