Return-Path: X-Original-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-callback-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 94A22D80C for ; Mon, 27 Aug 2012 17:24:12 +0000 (UTC) Received: (qmail 54663 invoked by uid 500); 27 Aug 2012 17:24:12 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 54635 invoked by uid 500); 27 Aug 2012 17:24:12 -0000 Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: callback-dev@incubator.apache.org Delivered-To: mailing list callback-dev@incubator.apache.org Received: (qmail 54624 invoked by uid 99); 27 Aug 2012 17:24:12 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 27 Aug 2012 17:24:12 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of purplecabbage@gmail.com designates 74.125.82.175 as permitted sender) Received: from [74.125.82.175] (HELO mail-we0-f175.google.com) (74.125.82.175) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 27 Aug 2012 17:24:06 +0000 Received: by weyr6 with SMTP id r6so2493357wey.6 for ; Mon, 27 Aug 2012 10:23:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=hhZH6Nk17z7T72sFjtqwytbDWeFqmV00u0HsUFq4zLg=; b=GnSRXO3bnhFze0W9Ii24U+AxoWKwFvodENkrgVgKtdeXZlm9oZntFBRkYr/EoKSw5X PU0nfVCmGw2DRXQUmI0FhM3pTx35APQaAvuy3D7kfBobdld49628hVV4zGfAbU/SVQjB MvSovdemWdVL7hmE65od5fAGIVuVeFWJEd8Huv8kMI2omLEzKvzLwdPuYHgv/jg5aNO1 z+1d2aMiVMOeE4qPvQGdgz4YBPOqMogovJTcyCY59xqpIrnG3iUr7KjpUdRDM9xwfMo5 siLzYzG9M3QJ6vlzww6yCKQ9IraeztznqPN7AbBIj/JbJ1zChOI2+3NEpd+tVG16dScL AwKQ== MIME-Version: 1.0 Received: by 10.180.97.33 with SMTP id dx1mr26833761wib.18.1346088226062; Mon, 27 Aug 2012 10:23:46 -0700 (PDT) Received: by 10.216.228.66 with HTTP; Mon, 27 Aug 2012 10:23:45 -0700 (PDT) In-Reply-To: References: Date: Mon, 27 Aug 2012 10:23:45 -0700 Message-ID: Subject: Re: Code Prettifiers From: Jesse To: callback-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=f46d04428f0c8d033b04c842965b --f46d04428f0c8d033b04c842965b Content-Type: text/plain; charset=ISO-8859-1 Focus on being a contributor and not an arbiter. If these problems do indeed become problems, then we will deal with them, however, I don't think we have issues currently, and even if we do, they are minor IMHO. + everything that Michael said, although, I would not shame anyone, and would simply weigh the value of the contribution with the work required to appease critical peers, before deciding to pull or push back. On Mon, Aug 27, 2012 at 10:12 AM, Patrick Mueller wrote: > On Mon, Aug 27, 2012 at 10:33 AM, Andrew Grieve >wrote: > > > ... > > > > One thing we may have to do is ask a contributor to enable the git hooks. > > AFAIK there is no way to enable shared git hooks for a project by > default, > > so what we'd have is a directory called "hooks" and you'd need to create > a > > symlink for it into your .git/ directory. > > > > Another option is to do this in some sort of continuous build environment. > Where would the "style enforcer" post it's messages though? Open bugs? > > If no one objects, then I'll pursue this today/tomorrow. My plan is to > > start with whitespace (sed or awk), get that working, and then see what > > x-platform open-source formatters exist out there. > > > > Presumably these hooks need to be portable to *nix and Windows (where mac > == *nix). Could we do them in node, without having to shell out to > command-line tools? Or maybe have the hooks be js that shell out to > cmd-line tools you'd need to have installed. > > -- > Patrick Mueller > http://muellerware.org > -- @purplecabbage risingj.com --f46d04428f0c8d033b04c842965b--