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

追加: CONTRIBUTING.md リリース #1119

Merged
merged 18 commits into from
Jun 20, 2024

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Mar 10, 2024

内容

README.mdCONTRIBUTING.md の重複を削除し、CONTRIBUTING.md を公式リリース(工事中 の削除)する。

関連 Issue

resolve #686 (final step 🎉)

@tarepan tarepan requested a review from a team as a code owner March 10, 2024 16:51
@tarepan tarepan requested review from Hiroshiba and removed request for a team March 10, 2024 16:51
Copy link

Coverage Result

Resultを開く
Name Stmts Miss Cover
run.py 513 218 coverage-58%
voicevox_engine/init.py 1 0 coverage-100%
voicevox_engine/cancellable_engine.py 94 72 coverage-23%
voicevox_engine/core/init.py 0 0 coverage-100%
voicevox_engine/core/core_adapter.py 81 12 coverage-85%
voicevox_engine/core/core_initializer.py 60 30 coverage-50%
voicevox_engine/core/core_wrapper.py 225 157 coverage-30%
voicevox_engine/dev/init.py 0 0 coverage-100%
voicevox_engine/dev/core/init.py 0 0 coverage-100%
voicevox_engine/dev/core/mock.py 65 4 coverage-94%
voicevox_engine/dev/tts_engine/init.py 0 0 coverage-100%
voicevox_engine/dev/tts_engine/mock.py 28 0 coverage-100%
voicevox_engine/engine_manifest/EngineManifest.py 36 0 coverage-100%
voicevox_engine/engine_manifest/EngineManifestLoader.py 12 0 coverage-100%
voicevox_engine/engine_manifest/init.py 0 0 coverage-100%
voicevox_engine/library_manager.py 92 4 coverage-96%
voicevox_engine/metas/Metas.py 36 0 coverage-100%
voicevox_engine/metas/MetasStore.py 28 1 coverage-96%
voicevox_engine/metas/init.py 0 0 coverage-100%
voicevox_engine/model.py 180 3 coverage-98%
voicevox_engine/morphing.py 71 4 coverage-94%
voicevox_engine/preset/Preset.py 13 0 coverage-100%
voicevox_engine/preset/PresetError.py 2 0 coverage-100%
voicevox_engine/preset/PresetManager.py 80 2 coverage-98%
voicevox_engine/preset/init.py 0 0 coverage-100%
voicevox_engine/setting/Setting.py 11 0 coverage-100%
voicevox_engine/setting/SettingLoader.py 17 0 coverage-100%
voicevox_engine/setting/init.py 0 0 coverage-100%
voicevox_engine/tts_pipeline/init.py 0 0 coverage-100%
voicevox_engine/tts_pipeline/kana_converter.py 88 1 coverage-99%
voicevox_engine/tts_pipeline/mora_mapping.py 7 0 coverage-100%
voicevox_engine/tts_pipeline/phoneme.py 34 0 coverage-100%
voicevox_engine/tts_pipeline/text_analyzer.py 146 6 coverage-96%
voicevox_engine/tts_pipeline/tts_engine.py 267 9 coverage-97%
voicevox_engine/user_dict/part_of_speech_data.py 5 0 coverage-100%
voicevox_engine/user_dict/user_dict.py 146 12 coverage-92%
voicevox_engine/utility/init.py 0 0 coverage-100%
voicevox_engine/utility/connect_base64_waves.py 37 0 coverage-100%
voicevox_engine/utility/core_utility.py 6 0 coverage-100%
voicevox_engine/utility/core_version_utility.py 8 1 coverage-88%
voicevox_engine/utility/mutex_utility.py 13 0 coverage-100%
voicevox_engine/utility/path_utility.py 26 6 coverage-77%
voicevox_engine/utility/run_utility.py 10 7 coverage-30%
TOTAL 2438 549 coverage-77%

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.

すみません、ちょっとリリースするにはCONTRIBUTING.mdのリファクタリングがもうちょっと必要かなと思いました!

まだパッと見ただけですが気になった点を列挙してみます 🙇

  • フォーマットされてない
  • 表記ゆれ(VOICEVOX/voicevox)
  • ####の数がおかしいときがありそう
  • 「インストール」項目でインストール案内していない(多分言いたいことは「環境構築」)
  • 「セットアップ」項目がかなり説明不足(「以下のコマンドで使用できるようになります。」で何が使用できるのかがわからない、とか)

あとREADMEから削除されてる部分で、このリポジトリをフォークして使う方向けの案内は残しておくと良いかもと思いました!
具体的には多分これだけ

  • 製品版のビルド方法
  • Github Actions周り(ビルド方法・変数の指定)
  • ライセンス周り

「詳しくは貢献者ガイドラインをご覧ください」とかでも問題ないかもです!

@tarepan tarepan marked this pull request as draft March 18, 2024 19:25
@tarepan
Copy link
Contributor Author

tarepan commented Jun 3, 2024

現状

#1215 により大幅ブラッシュアップが完了した。よって CONTRIBUTING.md 自体は正式リリース可能である。リリース時には工事中表記を外す必要がある。
全体としてみると、リリースまでに最後の詰めが必要である。具体的には README.mdCONTRIBUTING.md の重複がある。

対処

README.md 側の重複分を簡略化して案内とリンクへ置き換える。これにより重複を解消する。
重複を解消してもなお分量が多いため、各ガイドへの目次を冒頭へ追加する。
それと同時に CONTRIBUTING.md の工事中表記を削除する。
これにより CONTRIBUTING.md が正式リリースとなる。


@Hiroshiba
最終 PR が完成しました。レビューよろしくお願いします。

@tarepan tarepan marked this pull request as ready for review June 3, 2024 12:47
@tarepan tarepan requested a review from a team as a code owner June 3, 2024 12:47
@tarepan tarepan requested review from Hiroshiba and removed request for a team June 3, 2024 12:47
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です!!

すみませんお待たせしました!!

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@tarepan
Copy link
Contributor Author

tarepan commented Jun 20, 2024

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-review よろしくお願いします。

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!!

1箇所だけ変えて、こちらでフォーマットしてマージします!!

すごく丁寧な玄関ができたと思います!!ありがとうございました!!!!!!!!!!

README.md Outdated Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit 026bda6 into VOICEVOX:master Jun 20, 2024
4 checks passed
@Hiroshiba
Copy link
Member

@tarepan あ、1つご相談が!

ニーズのある機能が実装されたときにSNSで言及しておりまして、今回のプルリクエストもツイートしたいと思っています。
https://x.com/search?q=%23VOICEVOX%E9%96%8B%E7%99%BA%E7%8A%B6%E6%B3%81
もしよかったらそこで @tarepan さんのXアカウントをツイート文に含めて紹介させていただきたいのですが、構わないでしょうか・・・?

こんな感じを予定しています・・・!

#VOICEVOX開発状況 
VOICEVOXエンジンの貢献者ガイドラインができました🎉
プルリクエストレビューの流れやビルド方法まで一通り紹介しています。
ガイドラインはこちら↓
https://github.com/VOICEVOX/voicevox_engine/blob/main/CONTRIBUTING.md
【開発者: @tarepan_mhhow】
https://github.com/VOICEVOX/voicevox_engine/pull/1119

@tarepan tarepan deleted the add/contrib_prod branch June 21, 2024 02:24
@tarepan
Copy link
Contributor Author

tarepan commented Jun 21, 2024

紹介させていただきたいのですが、構わないでしょうか

OKです👍️

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