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

chore: FlatConfigにしたら出るエラーをいくつか解消 #2564

Conversation

Hiroshiba
Copy link
Member

内容

@MT224244 さんが

でeslintをflat configにしてくださっています。
このPRをマージするといくつかエラーが出るので、それだけ解決しました。

一旦mainブランチに向けてPR出してますが、mainブランチでエラーになりそうだったら #2560 に向けてPR出そうと思います。

関連 Issue

ref #2560

その他

@Hiroshiba Hiroshiba requested a review from a team as a code owner February 25, 2025 09:29
@voicevox-preview-pages
Copy link

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:b53801f

Comment on lines -1548 to +1551
const base64Encoder = (blob: Blob): Promise<string | undefined> => {
return new Promise((resolve, reject) => {
const reader = new FileReader();
reader.onload = () => {
// string/undefined以外が来ることはないと思うが、型定義的にArrayBufferも来るので、toStringする
const result = reader.result?.toString();
if (result) {
// resultの中身は、"data:audio/wav;base64,<content>"という形なので、カンマ以降を抜き出す
resolve(result.slice(result.indexOf(",") + 1));
} else {
reject();
}
};
reader.readAsDataURL(blob);
});
const base64Encoder = async (blob: Blob): Promise<string> => {
const arrayBuffer = await blob.arrayBuffer();
return toBase64(new Uint8Array(arrayBuffer));
Copy link
Member Author

Choose a reason for hiding this comment

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

fast-base64使って簡単にしました。
一応これでここのコード(音声を全て繋げて保存)が動くことは確認しました。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Feb 26, 2025

@sevenc-nanashi
すみません、ちょっと念のため見ていただけると・・・!!
(流石に大丈夫だと思うのですが。。。)

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

特に問題なさそう。

@Hiroshiba
Copy link
Member Author

レビューありがとうございます!! マージします!!

@Hiroshiba Hiroshiba enabled auto-merge February 26, 2025 23:08
@Hiroshiba Hiroshiba added this pull request to the merge queue Feb 26, 2025
Merged via the queue into VOICEVOX:main with commit 6bb9883 Feb 26, 2025
11 checks passed
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.

2 participants