All,

Over the past few days, i have made several commits to the repository
without code review.  Our convention has historically been to perform a
code review for any change, however small.  Please see below for some
rationale, but i would like to propose that we allow committers to exercise
judgement on skipping code reviews for changes unrelated to build or test
of the main project (e.g. scheduler, executor, client, packaging).  What do
you all think?

As an example, i think the code review process is too much overhead for
commits like the ones below.  With these commits i was playing whack-a-mole
to get alignment between markdown rendering on github.com/apache/aurora and
aurora.apache.org.  Skipping code review allowed me to fix things in a much
shorter timeframe.

commit 0d9fe18
Author: Bill Farner <wfar...@apache.org>
Date:   Wed Dec 23 08:31:27 2015 -0800

    Fix string interpolation for release email.

commit df5200b
Author: Bill Farner <wfar...@apache.org>
Date:   Mon Dec 21 14:19:48 2015 -0800

    Fix formatting and work around anchor link issues in installing.md

commit 21c605e
Author: Bill Farner <wfar...@apache.org>
Date:   Mon Dec 21 14:11:10 2015 -0800

    Fix anchor links in installing.md.

commit 9326fa6
Author: Bill Farner <wfar...@apache.org>
Date:   Mon Dec 21 12:21:37 2015 -0800

    Link to install guide from docs/README.md

commit f8e59a4
Author: Bill Farner <wfar...@apache.org>
Date:   Mon Dec 21 12:12:56 2015 -0800

    Fix formatting issues in installing doc.

Reply via email to