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

docs: README, CONTRIBUTINGの修正 #2530

Merged
merged 4 commits into from
Feb 19, 2025
Merged

Conversation

nanae772
Copy link
Contributor

@nanae772 nanae772 commented Feb 12, 2025

内容

README.md, CONTRIBUTING.md に対して以下の修正を行いました。

関連 Issue

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

その他

@nanae772 nanae772 requested a review from a team as a code owner February 12, 2025 10:02
@nanae772 nanae772 requested review from Hiroshiba and removed request for a team February 12, 2025 10:02
@nanae772 nanae772 changed the title docs: README, CONTRIBUTINGの docs: README, CONTRIBUTINGの修正 Feb 12, 2025
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.

LGTM!

@voicevox-preview-pages
Copy link

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

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

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 enabled auto-merge February 19, 2025 15:03
@Hiroshiba Hiroshiba added this pull request to the merge queue Feb 19, 2025
Merged via the queue into VOICEVOX:main with commit d089f41 Feb 19, 2025
11 checks passed
@nanae772 nanae772 deleted the update-documents branch February 19, 2025 23:51
@nanae772
Copy link
Contributor Author

@Hiroshiba
レビュー&マージありがとうございます。
興味のある分野についてはざっくり以下のような感じです。

  1. coreの開発(Rust勉強中+合成音声生成のロジックを知りたい)
  2. CI/CDやテスト、リファクタリングなど開発生産性向上に関わること全般

エディタ(フロント)の開発もユーザーが直接触る部分ということで興味はありますが、
Vue.jsやelectronなどの知識が全く無いので今のところ貢献できる部分はあんまり無いかもです。
(上であげた2つも全然初心者レベルなのですが)

なのでそういった初心者レベルのもので自分が取り組めそうなものあれば、
ちょこちょこやっていきながら理解を深めていければよいなと思っています。

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 20, 2025

なるほどです!!!

コアだと結構プログラミングの知識というか、その周りの知識が必要ですが、
pip installで入るvoicevox_coreライブラリの作成とかは面白いかもです。
VOICEVOX/voicevox_core#653
(結構いろんな調整とかも大変だと思うので、もしご興味あればという感じ!)

エンジンとコアはそこそこ整頓されているので、issueから興味あるの探してみるのが割とおすすめです!
コアはちょっと今激変中なのでもう少し待った後だと手をつけやすいかも。
エンジンの方は割と安定中なので、issue見て良さそうなのあればぜひぜひ!!!

リファクタリングやテストだと、エディタが一番アツい感じではあります。ただスキルセット的に勉強や慣れは必要そう。
結構勉強するところからになってしまうとは思うのですが、UIを自動で動かしてテストするe2e(end-to-end)テストとかを足しまくるのとか、開発陣的にはすごく嬉しいです。
この辺りの自動テストが読みやすいかも。

test("ダブルクリックで歌詞を編集できる", async ({ page }) => {
await navigateToSong(page);
const sequencer = page.getByLabel("シーケンサ");
const getCurrentNoteLyric = async () =>
await sequencer.locator(".note-lyric").first().textContent();
// ノートを追加し、表示されるまで待つ
await sequencer.click({ position: { x: 107, y: 171 } });
await page.waitForSelector(".note");
// ノートの歌詞を取得
const note = sequencer.locator(".note").first();
const beforeLyric = await getCurrentNoteLyric();
// ノートをダブルクリックし、入力フィールドが表示されるまで待つ
await note.dblclick();
await page.waitForSelector(".lyric-input");
// 歌詞を入力し、Enterキーを押す
const lyricInput = sequencer.locator(".lyric-input");
await lyricInput.fill("あ");
await lyricInput.press("Enter");
// 変更が反映されるまで待つ
await page.waitForFunction(() => {
const lyricElement = document.querySelector(".note-lyric");
return lyricElement && lyricElement.textContent === "あ";
});
// 歌詞が変更されたことを確認
const afterLyric = await getCurrentNoteLyric();
expect(afterLyric).not.toEqual(beforeLyric);
expect(afterLyric).toEqual("あ");
});

いくつか頂いたプルリクエストを見た感じ、一番手をつけやすいのはエンジンのissueだと思います!!
紹介に興味がある点あればそちらもぜひ!!!

@nanae772
Copy link
Contributor Author

いろいろと教えていただきありがとうございます。
まずはエンジンやコア周りで私ができそうなissueあればやってみようと思います。
フロントもvueやelectron周り勉強しながら、テストやリファクタリングで手をつけられそうなものあればやってみます!

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 22, 2025

フロントもvueやelectron周り勉強しながら、テストやリファクタリングで手をつけられそうなものあればやってみます!

こちらもよければ是非!!
ちなみにVueはドキュメントが整備されていて勉強しやすいと思います。
electronは細かいものが多岐にわたってるので勉強は難しいかもです。先にコードを少し眺める方が分かりやすいかも。
あとテストはPlaywrightやvitestという別のを使っているので、こっちはこっちで少し勉強がいるかもです。
けどまあVue以外はどれもこれも既存コードちょっと眺めると意外とつかめると思います!

不明な点などあればissueとかでコメントいただければ!

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.

3 participants