Skip to content

Instantly share code, notes, and snippets.

Show Gist options
  • Save qnighy/25c4d8ed293ec96a4f30ee45a6998875 to your computer and use it in GitHub Desktop.
Save qnighy/25c4d8ed293ec96a4f30ee45a6998875 to your computer and use it in GitHub Desktop.

SATySFiコミッターポリシー提案

用語

  • gfnさん = gfnさん
  • コミッター = コミット権限のある人
  • コントリビューター = SATySFiへのPRを出している人・出した人

PRについて

  • PRはgfnさんまたはコミッターによるレビューが必須 (レビュー必須の設定にしておく)
  • コミッターはmasterに直接pushせず必ずPRを出す。(別のコミッターのレビューを受ける)

変更点について

  • バックエンドに関してはリファクタリングも含め、触らない。どうしても必要なら作業前にgfnさんに相談するのが望ましい
  • 明らかな破壊的変更に関しては、マージする前にgfnさんの確認をとる。
  • 破壊的変更かどうか怪しい場合も確認をとる。たまった知見はできる限り共有する。
  • 破壊的変更だと後からわかる場合もある。その場合はその時に必要なアクションをとる。「とりあえずrevert」してもよいし、緊急度が高くなさそうなら一旦gfnさんの判断を聞いてみてもよい。
  • 破壊的でない変更についても、設計上の決断が必要なものは破壊的変更に準じる。
    • たとえば極端な例として、「stdjabookにTikZのような図形描画機能を加える」というのが考えられる。これは非破壊的だがおそらく「パッケージの趣旨に反する」と判断される。
    • 逆に、「不足しているギリシャ文字を追加する」などであれば設計上の決断は不要とも見なせる。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment