Update: 依存関係を更新する #2360
Update: 依存関係を更新する #2360
Conversation
sevenc-nanashi
requested review from
Hiroshiba and
Copilot
and removed request for
a team
There was a problem hiding this comment.
Copilot reviewed 8 out of 20 changed files in this pull request and generated 2 suggestions.
Files not reviewed (12)
- .node-version: Language not supported
- package.json: Language not supported
- public/licenses.json: Language not supported
- src/styles/_index.scss: Language not supported
- src/styles/v2/colors.scss: Language not supported
- build/download7z.js: Evaluated as low risk
- build/generateLicenses.js: Evaluated as low risk
- src/components/Sing/ToolBar/ToolBar.vue: Evaluated as low risk
- src/backend/electron/main.ts: Evaluated as low risk
- src/backend/electron/manager/vvppManager.ts: Evaluated as low risk
- src/backend/browser/sandbox.ts: Evaluated as low risk
- src/backend/electron/ipc.ts: Evaluated as low risk
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
| @@ -7,7 +7,6 @@ const config: StorybookConfig = { | |||
| "@storybook/addon-links", | |||
| "@storybook/addon-essentials", | |||
| "@chromatic-com/storybook", | |||
| "@storybook/addon-interactions", | |||
There was a problem hiding this comment.
experimental-addon-testと競合するので消しました。
info => Starting manager..
AddonConflictError: You have both "@storybook/addon-interactions" and "@storybook/experimental-addon-test" listed as addons in your Storybook config. This
is not allowed, as @storybook/experimental-addon-test is a replacement for @storybook/addon-interactions.
Please remove "@storybook/addon-interactions" from the addons array in your main Storybook config at ./.storybook and remove the dependency from your pack
age.json file.
src/styles/v2/colors.scss
Outdated
There was a problem hiding this comment.
ここら辺はsassの自動マイグレーションでなんとかしました。
There was a problem hiding this comment.
Copilot reviewed 8 out of 16 changed files in this pull request and generated no suggestions.
Files not reviewed (8)
- .node-version: Language not supported
- package.json: Language not supported
- public/licenses.json: Language not supported
- src/styles/_index.scss: Language not supported
- src/styles/v2/colors.scss: Language not supported
- src/components/Sing/ToolBar/ToolBar.vue: Evaluated as low risk
- src/backend/electron/main.ts: Evaluated as low risk
- src/backend/electron/manager/vvppManager.ts: Evaluated as low risk
There was a problem hiding this comment.
LGTM！！
tsconfig.jsonだけ確認のためにお待ちしてマージしたいと思います 🙏
electronのリリースノートを確認したのでメモ。
一言でいうと問題なさそう。
32のときにフロント側の File.path が削除されたとのこと。
もしかしたらどこかで影響してるかもだけど大丈夫な気がする。
33のときにmacはmacOS 11 (Big Sur)以上になったっぽい。
けどまあVOICEVOXはmacOS 12以上なので問題なし。
scriptsの変更戻していただいてありがとうございます 🙇
今のbuildにいろいろありすぎというのは同感です！！
なんとなくtools/scriptsあたりに移動するのが良いのかなと思っています。
こうすればvendoredはtools/vendoredに移動できて自然なので。
src/styles/v2/colors.scss
Outdated
There was a problem hiding this comment.
@romot-co 共有です！
sassのlightenなどは最近のバージョンでは無いらしく、案内に従ってcolor.adjustに変更されます！
https://sass-lang.com/documentation/modules/color/#lighten
ちなみに↑のドキュメントを見るに「color.scaleのが望ましい」的な事が書いているので、将来的にscaleに置き換えてもよいのかもとか思いました！
ちなみに自動マイグレーション機能とのこと。便利。
#2360 (comment)
There was a problem hiding this comment.
@Hiroshiba
共有ありがとうございます！
一応確認してみたものの、ソング関係はSASSのcolor関連機能は使っていないのであまり影響ないかもです(少なくともlighten/darkenなかった
もしかするとtakuseaさんに作成いただいている方に影響あるかも…？
@takusea
影響あるかもなので念のため共有になります…！🙇
src/backend/electron/ipc.ts
Outdated
There was a problem hiding this comment.
@sabonerune
IPCの不正チェック周りのコードがちょっと変更されるので共有です 🙏
Co-authored-by: Hiroshiba <[email protected]>
内容
関連 Issue
（なし）
スクリーンショット・動画など
（なし）
その他
（なし）