This is where i look towards allowing committers to exercise judgement. IMHO for commits like the ones above, a review is extra noise for everyone. I suppose my position is that i favor simplifying commits over simplifying comments.
On Thu, Dec 24, 2015 at 9:12 AM, Joshua Cohen <jco...@apache.org> wrote: > It could be, I just think it's easier to comment on a reviewboard than it > is a commits@ email. > > On Thu, Dec 24, 2015 at 10:29 AM, Bill Farner <wfar...@apache.org> wrote: > > > Can that be handled by subscribing to commits@? > > > > On Thursday, December 24, 2015, Joshua Cohen <jco...@apache.org> wrote: > > > > > I'm generally ok with this. Just curious: what do you think about maybe > > > posting a review and then committing it right away in these cases > > though? A > > > bit noisy on the reviews@ list, but at least it'd give people a chance > > to > > > peruse/comment as they see fit (with the assumption that any comments > > would > > > be followed up in a subsequent commit if needed). > > > > > > On Wed, Dec 23, 2015 at 3:48 PM, Bill Farner <wfar...@apache.org > > > <javascript:;>> wrote: > > > > > > > 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 <javascript:;>> > > > > Date: Wed Dec 23 08:31:27 2015 -0800 > > > > > > > > Fix string interpolation for release email. > > > > > > > > commit df5200b > > > > Author: Bill Farner <wfar...@apache.org <javascript:;>> > > > > 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 <javascript:;>> > > > > Date: Mon Dec 21 14:11:10 2015 -0800 > > > > > > > > Fix anchor links in installing.md. > > > > > > > > commit 9326fa6 > > > > Author: Bill Farner <wfar...@apache.org <javascript:;>> > > > > 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 <javascript:;>> > > > > Date: Mon Dec 21 12:12:56 2015 -0800 > > > > > > > > Fix formatting issues in installing doc. > > > > > > > > > >