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

マルチトラック:mainをマージ #2125

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Jun 19, 2024

内容

project-multitrackのbufferにmainをマージします。

関連 Issue

(なし)

スクリーンショット・動画など

(なし)

その他

(なし)

weweweok and others added 30 commits May 21, 2024 03:54
fix: テキスト読み込みルビ文字列を削除できるように
* perf: BASE64のデコードを`Buffer.from`から`atob`へ移行

* fix: `imageHelper`to`base64Helper`

* perf: `buffer`に戻す
* 辞書保存ダイアログの編集破棄の案内がわかりにくかったので修正

* 他にもわかりにくいとこ直した
* グリッドをコンポーネント化した

* 最後の小節でマウスホイールで縮小を行ったときに表示がおかしくなるのを修正

* PreviewModeの文字列を変更
* projectのschemaをdomainディレクトリ以下に移動

* 移動し忘れ

* noteSchema移動し忘れ

* たぶんこう

* CharacterInfo差し戻し

* anyは仕方ない気がする

* import ミス

* ミス

* ProjectFileFormatError型を用意

* causeいらない
ソングのデフォルトスタイルは一旦実装しない方針
* feat: 登録単語の編集・削除ボタンの常時表示をやめてOOUI的に

* fix: ツールチップをQTooltipに直す

* fix: 一応削除ボタンのclickイベントハンドラに `.stop` を追加

* test: 辞書ダイアログのe2eテストを更新

* feat: 選択時だけでなくホバー時にもボタンを表示する

* fix: 辞書アイテム、編集を左に削除を右に

* fix: 削除ボタンのスタイルをAudioCell のそれに合わせる

* refactor: ホバー状態を単純な文字列で管理
* Migrate: ESModuleに移行

* Revert: vuexの変更をRevert

* Fix: __dirnameを追加

* Fix: electronが立ち上がらないのを修正

* Change: type: moduleをやめる

* Delete: __dirnameをなくす

* Revert: js -> cjsをやめる

* Code: コメントを追加

* Revert: package.jsonのパスも戻す

* Revert: 色々なコンフィルファイルの拡張子も戻す

* Revert: electron-builder.config.jsのも戻す

* Revert: js内のも戻す
* 全選択ショートカットキーをctrl+Aに変更する

* 有効なときのみ実行するようにした
* Test: 設定ダイアログのスクリーンショットテスト

* [update snapshots]

* (スナップショットを更新)

* テストのための空コミット

* fullPage [update snapshots]

* add .png [update snapshots]

* (スナップショットを更新)

* Revert "(スナップショットを更新)"

This reverts commit 62f51a9.

* Revert "add .png [update snapshots]"

This reverts commit f8317dc.

* Revert "fullPage [update snapshots]"

This reverts commit efb475d.

* [update snapshots]

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* no-unused-varをビルド時エラーに

* 修正
VOICEVOX#2082 tempファイル作成後に設定ファイルを書き込むよう修正
* add feature dot notation action

* use dot notation store in dictionary.ts

* このPRと直接関係ないがvuex.tsのStoreはclassである必要がないためfix

* add mutaitons and actions to Store class

* import alias `createDotNotationPartialStore` to `createDotPartialStore`

* コメント追加とDispatch等をstore内でも見えるように

* fix name DotPartStore

* 丁寧語になっていたのを修正

* as unknownの文についてFIXMEを追加

* rename import  `createPartialStore`

* format
* ユーザーが描いたピッチがずれるのを修正、リファクタリング

* ピッチ編集モードの時にノートの端や縁の色が変わらないようにした
* Change: fast-base64を使う

* Refactor: asyncComputedに統一

* Improve: キャッシュを追加

* Change: useEngineIconsにする

* Code: コメントを足す

* Fix: 代入忘れ

* Change: computedにする

* Fix: 型を足す

* Add: コメントを追加

* Update src/components/Menu/MenuBar/MenuBar.vue

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* プロジェクトファイルのマイグレーションのテストを追加

* japanese domain

* 忘れ

* テスト更新

* fmt
* Test: 設定ダイアログのスクリーンショットテスト

* Toggleを共通化

* [update snapshots]

* (スナップショットを更新)

* テストのための空コミット

* fullPage [update snapshots]

* add .png [update snapshots]

* (スナップショットを更新)

* Revert "(スナップショットを更新)"

This reverts commit 62f51a9.

* Revert "add .png [update snapshots]"

This reverts commit f8317dc.

* Revert "fullPage [update snapshots]"

This reverts commit efb475d.

* [update snapshots]

* 文言調整

* fmt

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Bump actions

* "crate-ci/typos action"のエラーを修正

* Apply suggestions from code review

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* vue/attribute-hyphenation: never

* fmt on vue/attribute-hyphenation: never

* introduce propert-shorthand rule

* fmt on @voicevox/propert-shorthand

* 最新版のeslint

* checkout main

* Refactor: Update component prop names to adhere to Vue.js style guide

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* darwin-arm64プラットフォーム対応

* アクションのmacOSビルドでアドホック署名

* macOSの`.app`ファイルが正しい構造を持つように修正

Contents/MacOSフォルダには実行ファイルのみが存在するべきですが、
その他のファイルも含まれていたため、正しい構造ではなく、
コード署名が失敗しました。

* テストからmacOS arm64を一時的に除外

Co-Authored-By: Hiroshiba <hihokaruta@gmail.com>

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* 途中

* できた

* @update:model-valueに

* slot

* ちょっと変更

* npm run fmt
* Add: ブラウザ版にファイル読み込みを追加

* Code: 意図を追加

* Change: uuidを足す

* Code: コメントを足す

* Refactor: checkOsにまとめる

* Change: electron側と合わせる

* Add: 型を足す

* Change: パスの形式を変える

* Add: ファイルパスをエラーに追加

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Code: メモを追加

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Apply suggestions from code review

---------

Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* 使い方に歌詞の入力を追加

* Apply suggestions from code review
* Add: @sevenc-nanashi/utaformatix-tsを入れる

* Change: ImportExternalProjectDialogにする

* Change: IMPORT_MIDI_FILEをIMPORT_EXTERNAL_PROJECT_FILEに

* Add: 歌詞を変換する

* Delete: midi-fileを依存から消す

* Add: parseFailedのメッセージを追加

* Change: convertVowelConnectionsを有効にする

* Add: vvprojを読み込めるように

* Change: externalSongProjectにする

* Change: 残っていたmidiを変える

* Change: 0.2.3に更新

* Fix: ファイル名を修正

* Fix: 型エラーを修正

* Code: フォーマット

* Add: 拡張子一覧を追加

* Update: DIの方を更新

* Change: プロジェクトファイルの順番を変える

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>

* Change: CeVIO AI -> CeVIO

* Refactor: projectFileErrorを良い感じにする

* Change: SMFの表記を変える

* Change: details-summaryで囲む

* Change: Project -> UfProject

* Change: PARSE_PROJECT_FILEに移動

* Change: ufDataの変換を別ファイルに移動

* Add: テストを追加

* Change: midi -> ufProjectToSongState

* Fix: Add忘れ

* Change: log.errorの引数を変える

* Update: パスを更新

* Code: コメントを追加

* Change: プロジェクトファイルのインポート -> インポート

* Refactor: 整理

* Fix: パスを更新

* Add: vvprojの読み込みお変える

* Update: Update utaformatix-ts

* Update: 本家の色々に追従

* Delete: 不要なフォールバックを削除

* Fix: 変換を修正

* Update: テスTを更新

* Change: uuidv4 -> crypto.randomUUID

* Change: コンバーターの名前を変える

* Change: SongState -> VoicevoxScore

* Code: npm run fmt

* Add: TODOを追加

* Code: TODOを追加

Co-Authored-By: Hiroshiba <Hiroshiba@users.noreply.github.com>

* Code: TODOを削除

* Code: コメントの解釈を一つに

* Code: TODOを追加

* Refactor: 良い感じにする

* Add: エクスポートのテストを追加

* 改変してみた

* Delete: 不要なroundを削除

* Delete: 不要なimportを削除

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <Hiroshiba@users.noreply.github.com>
nmori and others added 3 commits June 16, 2024 22:38
* 未対応エンジン追加時にリストが消える件(VOICEVOX#1168)
・追加されたエンジンが未対応である場合には追加を阻止
・追加されてしまっている場合には、エラーで処理中断しないように

* lintチェックエラー部分の修正

* コードレビューの反映 (ref VOICEVOX#1179)
・MinimumEngineManifestの更新

* コードレビュー分の反映② ref VOICEVOX#1179
・engineManifests[selectedId]自体が undefined であるケースに対応

* 貢献者ガイドラインを明文化 (ref VOICEVOX#1190)

* レビュー結果の反映①

* 着手周りの手順追記

* CONTRIBUTING.md として配置変更

* markdownlint のエラーを修正

* * ローカル実行時の markdownlint 検索範囲を修正

* Issueを閉じるタイミングを追記

* * ドラフトプルリクエストについての追記
* フォーマットの修正

* * プルリクエストの表記を英語に。
* WIPに付いてのトーンを弱めに。
* リンク切れの修正
* 「その他」の 追記

* * レビュー内容の反映

* * e2e部分の追記
* インデント修正

* 提案いただいた分のコミットと追記

* ・査読分の反映
・README.mdに誘導リンクを追加

* Apply suggestions from code review

* フォーマットを整える

* 崩れてしまった部分を戻す

* こう?

* なぜか * に戻っていた

* pythonはコメントアウトが // ではなかった

---------

Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
* シング側のポートレート位置を調整する

* 微調整

---------

Co-authored-by: Romot <user@USERs-MacBook-Pro.local>
Co-authored-by: Hiroshiba Kazuyuki <kazuyuki_hiroshiba@dwango.co.jp>
@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner June 19, 2024 21:26
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team June 19, 2024 21:26
@Hiroshiba Hiroshiba changed the base branch from buffer/project-multitrack/20240620 to project-multitrack June 20, 2024 01:48
@Hiroshiba
Copy link
Member

Hiroshiba commented Jun 20, 2024

あ、必要なのはproject-multitrackへのPRかも?

ということで宛先をproject-multitrackにしてみたらmergeできそう。
認識合ってそうだったらsquashなしマージします・・・!!

@sevenc-nanashi
Copy link
Member Author

あっ(bufferブランチいまだによくわかってない)
多分それであってます、よろしくお願いします。

@Hiroshiba
Copy link
Member

コンフリクトが発生していたなら「mainからbufferブランチを作ってproject-multitrackをマージし、project-multitrackに発射」かな・・・?(たぶん今ほぼその形になってる)
といっても僕も雰囲気でやってるので、なんでもよさそう!!
https://github.com/VOICEVOX/voicevox_project/blob/main/docs/%E3%83%96%E3%83%A9%E3%83%B3%E3%83%81%E6%88%A6%E7%95%A5.md#release%E3%83%96%E3%83%A9%E3%83%B3%E3%83%81%E3%82%92main%E3%83%96%E3%83%A9%E3%83%B3%E3%83%81%E3%81%AB%E3%83%9E%E3%83%BC%E3%82%B8%E3%81%99%E3%82%8B%E3%81%A8%E3%81%8D%E3%81%AB%E3%82%B3%E3%83%B3%E3%83%95%E3%83%AA%E3%82%AF%E3%83%88%E3%81%8C%E7%99%BA%E7%94%9F%E3%81%99%E3%82%8B%E5%A0%B4%E5%90%88

@Hiroshiba Hiroshiba merged commit 12abe27 into VOICEVOX:project-multitrack Jun 20, 2024
9 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.