Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Electron to v16 #495

Merged
merged 3 commits into from Nov 21, 2021
Merged

Conversation

koizuka
Copy link
Contributor

@koizuka koizuka commented Nov 20, 2021

内容

Electronのバージョンを最新の v16.0.1 に上げます。対応するSpectronは存在しないので、Spectronは削除します。

保守的にv13にあげるPR(#494)も出しましたが、問題が無ければこっちのほうが良いですね。

動作確認は軽くしかしていないので、チェックお願いします...

関連 Issue

close #488

参考

  • v14 の Breaking Changes
    • Child windows no longer inherit BrowserWindow construction options from their parents. #28550
    • Deprecated worldSafeExecuteJavaScript option was removed from webPreferences. #28456
    • Removed deprecated additionalFeatures property from new-window and did-create-window WebContents events. #28548
    • Removed the deprecated app.allowRendererProcessReuse and BrowserWindow affinity options. #26874
  • v15 の Breaking Changes
    • nativeWindowOpen: true is now the default. #28552
  • v16の Breaking Changes
    • The crashReporter API is now powered by Crashpad on Linux. #30278
    • Usage of the desktopCapturer.getSources API from the renderer process has been deprecated and will be removed. 30721

みたところ上記の機能を使っているコードは見つかりませんでした。

@Hiroshiba
Copy link
Member

Hiroshiba commented Nov 21, 2021

すみません、ちゃんと見ず https://github.com/Hiroshiba/voicevox/pull/494 をマージしてしまってコンフリクトが発生してしまいました。
resolveして頂けると・・・!

@koizuka
Copy link
Contributor Author

koizuka commented Nov 21, 2021

mainをmergeしてresolveしました~

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!

いろいろ見てみたのですが、左のとこのキャラクター表示のスタイルがちょっと崩れてそうでした。
image
(想定では画像の縦幅は変わらず、キャラ名の文字は左上に出ます)

が、一旦マージしても大丈夫だと判断したので、マージします!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Electronのバージョンを上げる
2 participants