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

追加: 貢献者ガイドブラッシュアップ #1215

Merged
merged 30 commits into from
Jun 2, 2024

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented May 5, 2024

内容

概要: 大幅ブラッシュアップ版の貢献者ガイドを追加

React の How to Contribute を参考にして CONTRIBUTING.md を大幅ブラッシュアップした。

関連 Issue

ref #1130 (comment)

@tarepan tarepan requested a review from a team as a code owner May 5, 2024 07:48
@tarepan tarepan requested review from Hiroshiba and removed request for a team May 5, 2024 07:48
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 Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@tarepan
Copy link
Contributor Author

tarepan commented May 10, 2024

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

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

tarepan commented May 15, 2024

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

@tarepan tarepan requested a review from Hiroshiba May 26, 2024 03:56
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側で重複してる部分を消して、工事中案内文を消して、CONTRIBUTOR.mdへのリンクを案内すればOKという認識です!
(重複部分はAPIユーザー向けのものもあるはずなので、そこはそのままが良さそう。具体的にはUTF8のとこ。)

エディタ側のガイドラインとの兼ね合い

@nmori さんがエディタ側で貢献者ガイドラインを書いてくださっています。

エディタガイドラインはGithubとGitの使い方に比重が置かれている感じです。
エディタは特に初学者の方(特にGithub初学の方)が多めだと思うので、特色が違う形で良いのかなと思ってます!

とりあえずご共有まで 🙏

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
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.

マージします!

@Hiroshiba Hiroshiba merged commit 4e8c905 into VOICEVOX:master Jun 2, 2024
4 checks passed
@tarepan tarepan deleted the refactor/brushup_contrib branch June 2, 2024 17:14
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