StorybookのVRTにファイル一覧の同期チェックを追加 #2385
Conversation
[update snapshots]
sevenc-nanashi
requested review from
Hiroshiba
and removed request for
a team
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
[update snapshots]
[update snapshots]
[update snapshots]
[update snapshots]
[update snapshots]
[update snapshots]
[update snapshots]
[update snapshots]
.github/workflows/test.yml
Outdated
| path: patches | ||
| path: /tmp/patches | ||
| merge-multiple: true | ||
|
|
||
| - name: Commit updated snapshots | ||
| id: commit-updated-snapshots | ||
| run: | | ||
| # パッチを適用 | ||
| for patch in patches/*.diff; do | ||
| for patch in /tmp/patches/*.diff; do | ||
| git apply --allow-empty $patch | ||
| rm $patch |
There was a problem hiding this comment.
（細かいけど）
このpatchesの移動も不要だったりしそう？
/tmp/、windowsでの挙動がよーわからんので避けたいモチベがほんの少しあるかも、くらいですが。
（あと差分がきれい！）
There was a problem hiding this comment.
個人的に、Gitのworkdirはあんまり汚したくないんですよね…特にコミットが絡んでくると尚更。
これはubuntu-latestで実行されるのでWindowsは無視していいと思います。
There was a problem hiding this comment.
なるほど。
tmpを使うのも万能じゃなくて、他でtmp使うものが同じファイル名で書き出してたときにわけのわからない挙動になりますね。
基本globalなものはすべて恐ろしく、避けたい。
あとtmpは結構特別な扱いになってることも多いので意外と厄介な印象です。
今はlinuxでしか使ってないけど、という感じ。気軽に使って痛い目見るやつ。
ぱっと見つけたmacの例 https://autopp-tech.hatenablog.com/entry/2020/07/12/220856
ただ作業スペースを汚したくないのもわかります･･･。
今回の場合、リポジトリルートにpatchesというファイルができてもバグりますね･･･。
voicevox-update-snapshots-patchesみたいな名前にし、ディレクトリ丸ごと.gitignoreに追加して使うというのはどうでしょう？
（voicevoxプレフィックス付けてるのは他で衝突しないようにくらいの意図です）
There was a problem hiding this comment.
うーん、多分プロジェクトルートに.actions-tmpみたいなフォルダを作ってignoreに入れて、それを共用で使うのが良さそうな気がします（.actions-tmp/patchesみたいな）
There was a problem hiding this comment.
それでも良いと思います！
名前はgithub actionsだとわかるようにしたいかもです。
There was a problem hiding this comment.
RUNNER_TEMPを使うようにしました（なんかあったな〜って記憶はあったけど出てこなかった）
There was a problem hiding this comment.
同じくです AIに思い出してもらってました
https://chatgpt.com/share/674f0852-bf00-8008-8028-d58103737724
| if (test.info().config.updateSnapshots === "all") { | ||
| for (const screenshot of screenshotPaths) { | ||
| if (!expectedScreenshots.includes(screenshot)) { | ||
| await fs.unlink(screenshot); | ||
| } | ||
| } | ||
| } else { | ||
| expect(screenshotPaths).toEqual(expectedScreenshots); | ||
| } |
There was a problem hiding this comment.
（前のコメントから移動）
削除した旨をログで出しつつ、あとファイルの一番上で案内も！ 🙏
|
ちょっと念の為最終確認したいので空打ちcommitさせていただきます 🙏 |
|
マージします！ 使い勝手の良さが更に増しましたね！！ |
内容
タイトル通りです。
関連 Issue
（なし）
スクリーンショット・動画など
（なし）
その他
（なし）