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

voicevox-clientのレポジトリリンク変更に伴うREADMEの事例紹介の変更 #635

Merged
merged 3 commits into from
Feb 25, 2024

Conversation

tuna2134
Copy link
Contributor

内容

voicevox-clientっという組織のもとで作るのでリンクを変えました

@tuna2134 tuna2134 requested a review from a team as a code owner March 16, 2023 07:17
@tuna2134 tuna2134 requested review from y-chan and removed request for a team March 16, 2023 07:17
@github-actions
Copy link

github-actions bot commented Mar 16, 2023

Coverage Result

Resultを開く
Name Stmts Miss Cover
run.py 513 277 coverage-46%
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 59 30 coverage-49%
voicevox_engine/core/core_wrapper.py 257 183 coverage-29%
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 6 coverage-79%
voicevox_engine/metas/init.py 0 0 coverage-100%
voicevox_engine/model.py 180 9 coverage-95%
voicevox_engine/morphing.py 71 46 coverage-35%
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_version_utility.py 8 1 coverage-88%
voicevox_engine/utility/mutex_utility.py 13 0 coverage-100%
voicevox_engine/utility/path_utility.py 26 8 coverage-69%
voicevox_engine/utility/run_utility.py 10 7 coverage-30%
TOTAL 2463 689 coverage-72%

README.md Outdated
@@ -544,7 +544,7 @@ VOICEVOX エディターにうまく読み込ませられないときは、エ

## 事例紹介

**[voicevox-client](https://github.com/tuna2134/voicevox-client) [@tuna2134](https://github.com/tuna2134)** ・・・ VOICEVOX ENGINE のためのPythonラッパー
**[voicevox-client](https://github.com/voicevox-client/python) [@voicevox-client](https://github.com/voicevox-client)** ・・・ VOICEVOX ENGINE のためのPythonラッパー
Copy link
Member

Choose a reason for hiding this comment

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

Pythonだけじゃなくいろんなプログラミング言語向けラッパーという感じですよね。
それが分かる文面に変えるのはどうでしょう。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

そうですね
しかし、他の言語を担当している方が今忙しいので、出せないのですよね、、、

Copy link
Member

Choose a reason for hiding this comment

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

リポジトリが移管されてもリダイレクトは効くので、完成するまで一旦待機にするか、そもそも完成してなくてもいいのでhttps://github.com/voicevox-clientを指定してしまい、VOICEVOX ENGINEの各言語向けAPIラッパーなどと書くのが良いかなと思いました...!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

そうですね、、、そうしときます

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.

プルリクエストありがとうございます!

PRのタイトルは、何をどうしたものなのかわかりやすいものにすると、他の方にわかりやすそうです。
タイトルは後からも変えられるので変更してみてください。

@tuna2134 tuna2134 changed the title リンク変わったので、こっちも変えた voicevox-clientのレポジストリリンク変わったので、こっちも変えました Mar 16, 2023
@tuna2134
Copy link
Contributor Author

プルリクエストありがとうございます!

PRのタイトルは、何をどうしたものなのかわかりやすいものにすると、他の方にわかりやすそうです。 タイトルは後からも変えられるので変更してみてください。

箇条書きに書いてすいません
変更しときました

@y-chan y-chan changed the title voicevox-clientのレポジストリリンク変わったので、こっちも変えました voicevox-clientのレポジトリリンク変更に伴うREADMEの事例紹介の変更 Mar 17, 2023
@y-chan
Copy link
Member

y-chan commented Mar 17, 2023

PRの題に指示語を使うのはわかりづらいのでやめた方が良いですね...!
こちらで適切な形に書き換えておきましたので、今後の参考にしていただければ...!

@tuna2134
Copy link
Contributor Author

わかりました、あまり慣れてなくて、、、すいません

Copy link
Contributor

@tarepan tarepan 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になっていますね。
good work!

@Hiroshiba
こちらのPR、レビュー可能な状態かと思います。

@Hiroshiba Hiroshiba removed the request for review from y-chan February 25, 2024 22:44
@Hiroshiba
Copy link
Member

マージします!!

@Hiroshiba Hiroshiba merged commit a386cae into VOICEVOX:master Feb 25, 2024
4 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.

4 participants