Proposal: change default snapshot filename (~breaking)
還沒有人認領這個 Issue。
評估
- 難度
- 4/5
- 預估耗時
- 3-5 天
- 新手友好度
- 52/100
- Issue 類型
- 功能
- 描述清晰度
- 基本清楚
- 活躍度
- 冷清
- 技術堆疊
- node.js
研究方向
Read the snapshot implementation linked as nodejs-loaders/…/test/snapshots.mjs and trace how the default filename is generated and existing files are read. The change is complete when new snapshots use the .snap.cjs name and the proposed handling of old .snapshot files is covered without breaking existing tests.
由索引模型根據 Issue 內容生成。
描述
Problem(s) to solve
The current default value of a snapshot filename is to append .snapshot to the name of generating file (aka a "test file"). For instance foo.test.js → foo.test.js.snapshot.
This breaks any IDE trying to help with the file.
A snapshot file is actually a CJS file.
Proposed solution(s)
Adjust the default filename to ${TEST_FILE_BASENAME}.snap.cjs (see nodejs-loaders/…/test/snapshots.ts).
Ex foo.test.js → foo.test.snap.cjs
IDEs recognise this and render the snapshot file as the CJS that it actually is.
We can mitigate a breaking change by detecting the old on read and renaming the file on disk whilst also outputting a message to the console.
- 主要語言
- 沒有語言資料
- 星號
- 3
- 分支
- 4
- PR 合併指標
- 30 天內沒有已合併 PR
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
nodejs/test-runner 的其他 Issue
-
難度 1/5 1 小時以內 新手友好度 1/100
nodejs/test-runner#46 ·
-
難度 1/5 1 小時以內 新手友好度 5/100
nodejs/test-runner#45 ·
-
難度 1/5 1 小時以內 新手友好度 10/100
nodejs/test-runner#44 ·
-
難度 1/5 1 小時以內 新手友好度 15/100
nodejs/test-runner#43 ·
-
難度 1/5 1 小時以內 新手友好度 15/100
nodejs/test-runner#42 ·
查看 nodejs/test-runner 的全部 Issue
相似的 Issue
-
ci-rerun: collect.py fails a whole pr-babysit cycle on one transient GitHub 5xx and never retries未關閉follow-up
難度 2/5 1-3 小時 新手友好度 62/100
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 62/100
REditorSupport/languageserver#833 ·
維護者通常 1 天內回覆
-
cmd/ebitenmobile: multi-language bindings run the complete generator twice可能已有人在做 @kumagi 今天認領。 未關閉os:macos performance
難度 2/5 1-3 小時 新手友好度 82/100
hajimehoshi/ebiten#4011 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 78/100
-
難度 2/5 1-3 小時 新手友好度 76/100
nolabs-ai/nono-packs#65 ·