差分: https://gist.github.com/kmaehashi/5947820/revisions
変更内容は以下の通り。いずれも、過去のトラブルの反省(KPT)を元に見直し。
- configure バリエーションの確認を明記。
- コーディングスタイルの変更確認を明記。
- 「./waf install でインストールされるヘッダに Jubatus 独自のマクロが含まれていない」を追加。
差分: https://gist.github.com/kmaehashi/5947820/revisions
変更内容は以下の通り。いずれも、過去のトラブルの反省(KPT)を元に見直し。
Pull-Request の取扱いについて定める。なお、Pull-Request には Issue ハンドリングポリシも適用される (ここでは Pull-Request に特有の事項を述べる)。
全体
品質
個別観点 (注1)
Travis CI
(注1) 個別観点は、各リリースで行う振り返りのプロセスから適宜抽出して追加することで、発生しがちな問題を共有して再発を防止する。cf. Scalaのレビュー観点, Hadoopのレビュー観点
@unnonouno ./waf cpplint は自動生成されたコードをブラックリストで無視するようになっています。
LGTM
個別観点 (注1)
./waf install でインストールされるヘッダに Jubatus 独自のマクロが含まれていない
これは厳しすぎる or 現状でも守れていないのでは?
個別観点 (注1)
./waf install でインストールされるヘッダに Jubatus 独自のマクロが含まれていないこれは厳しすぎる or 現状でも守れていないのでは?
こちらは、マクロ = wscript で定義される定数(HAVE_ZOOKEEPER_H など)というニュアンスでした(現状守れていると思っています)。文言を直します。
server周りに修正があった時に、cpplintにはじかれることがあります。その場合、cpplint側の無視条件を修正?