Skip to content

Commit

Permalink
memberがPRをマージしてよい条件を明示的に制定 (#500)
Browse files Browse the repository at this point in the history
* Create contribution/index.md

* Update index.md
  • Loading branch information
FineArchs authored Dec 4, 2023
1 parent 0d57639 commit 905637e
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions docs/contribution/index.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
## PRマージ時の規則
以降の規則は[member](https://github.com/orgs/aiscript-dev/people)が何をしてよいか明示するものであり、何かを禁止するものではありません。(禁止のための条項が必要になれば別に作ります)
- バグ修正、ドキュメントの編集、dependabotのPRはmember1人の判断でマージしてよい。
- ソースコード及びテストの変更は、
- 非破壊的変更であればmember2人以上の同意&1日の経過を待ち、非memberを含む反対者が同意者の半数以下ならマージしてよい
- 破壊的変更であればmember3人以上の同意&1日の経過を待ち、非memberを含む反対者が同意者の4分の1以下ならマージしてよい
- ソースコード・テスト以外の変更はmember1人以上の同意&1日の経過を待ち、非memberを含む反対者が同意者の半数以下ならマージしてよい
- リポジトリの所有者(@syuilo)の同意がある場合、以上の規則によらずにマージしてもよい
- 大前提としてすべてのマージはCI Checkを通過し、全てのreviewにPR主から何らかの返答がある必要がある
- 後でrevertになっても泣かない

0 comments on commit 905637e

Please sign in to comment.