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

0.4.1のプロジェクトが0.9.3で開けないバグを修正 #595

Merged
merged 2 commits into from
Dec 15, 2021

Conversation

yuqquu
Copy link
Contributor

@yuqquu yuqquu commented Dec 14, 2021

内容

2点です。

  • (タイトル通り) 0.4.1で保存されたプロジェクトファイル(.vvproj)が0.9.3で開けないバグを修正
  • undefinedの判定を厳密化

関連 Issue

close #592

その他

0.4.1、0.5.3、0.6.1、0.7.5、0.8.2、0.9.3で保存されたプロジェクトが開発バージョンで正常に読み込まれることを確認しました。

Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

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

内容はLGTMです...!
ただ、後々のために(なぜここだけcharacterIndexなのか)このあたりにコメントを書いておいてもらえないでしょうか...?(日本語でも英語でも、わかりやすいように残しておいてもらえると大変ありがたいです...!)

@yuqquu
Copy link
Contributor Author

yuqquu commented Dec 15, 2021

レビューありがとうございます!
確かに、あとから変更されて再発しそうですね。
コメントを加えました。

Copy link
Member

@y-chan y-chan left a comment

Choose a reason for hiding this comment

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

LGTMです、ありがとうございます!

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.

LGTM!!
(issueにて確認頂くご連絡をお願いできるととても嬉しいです・・・!)

@Hiroshiba Hiroshiba merged commit 4300d9d into VOICEVOX:main Dec 15, 2021
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.

Ver0.4.1で保存したプロジェクト(.vvproj)がVer0.9.3で読めない
3 participants