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 EB2C4D43C for ; Tue, 9 Oct 2012 22:15:15 +0000 (UTC) Received: (qmail 26405 invoked by uid 500); 9 Oct 2012 22:15:15 -0000 Delivered-To: apmail-incubator-callback-dev-archive@incubator.apache.org Received: (qmail 26377 invoked by uid 500); 9 Oct 2012 22:15:15 -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 26364 invoked by uid 99); 9 Oct 2012 22:15:15 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 09 Oct 2012 22:15:15 +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 pmuellr@gmail.com designates 209.85.220.175 as permitted sender) Received: from [209.85.220.175] (HELO mail-vc0-f175.google.com) (209.85.220.175) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 09 Oct 2012 22:15:10 +0000 Received: by mail-vc0-f175.google.com with SMTP id p1so7580026vcq.6 for ; Tue, 09 Oct 2012 15:14:49 -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=mbpnOp1jvEOBc9DX/Ldl+qaABGw4iW8xSsR7C0hlRQs=; b=xD/sO+GXkWRtGntirjjdpuP42Rj339dyDiufQXpXY1+5UR8s4Cqs5PDlpjHGsNxZ+S nXBePR3Wkxfcu0XofRs11wvYlsN7nhO+jh8Jxo2IWkEMhRTA56phQWJW6JXrmuLm17wW iScbH5nBjCg1oQBFnDPqm9I2HEK5aYpBil9Kstm8tFTnbSjN8rM5hPFBVmsnRW45gweO boPL1KVU7hsjIQ3LdwQe/T9XDP1hkRuCg2gFryyl3uPxJVIinXu0megD/KOryZ0UwfWW LxFSy5R/w0pKtx0j1nQFIqpjN3vA+RXEjCMQ2r/AperOc87oD4rkAIY+v+dhDgEvQn5+ FqvA== MIME-Version: 1.0 Received: by 10.220.221.203 with SMTP id id11mr12494293vcb.42.1349820889140; Tue, 09 Oct 2012 15:14:49 -0700 (PDT) Received: by 10.58.125.39 with HTTP; Tue, 9 Oct 2012 15:14:49 -0700 (PDT) In-Reply-To: References: Date: Tue, 9 Oct 2012 18:14:49 -0400 Message-ID: Subject: Re: [DISCUSS] cordova.js and lint errors From: Patrick Mueller To: callback-dev@incubator.apache.org Content-Type: multipart/alternative; boundary=14dae9ccdd4e9b8f2a04cba7aa26 --14dae9ccdd4e9b8f2a04cba7aa26 Content-Type: text/plain; charset=ISO-8859-1 Add whatever linter to dependencies in package.json, and call it before every build, from it's node_modules/.bin script. Isn't it lightning quick, just spews loads of messages? On Tue, Oct 9, 2012 at 5:47 PM, Andrew Grieve wrote: > I've been using a pre-commit check on the iOS repo to make sure I don't > check in crusty code, and I've really liked how this has gone. I get a > warm-and-fuzzy feeling from having a tool check my code before it gets > committed. > > My vote would be for more of this. E.g. create a hooks/pre-commit file, and > change the README.md to tell people to symlink it into their .git > directory. > I'd also like to see this enforce change descriptions having [platform] in > it, or else even the most diligent of us will eventually forget. > > > On Tue, Oct 9, 2012 at 5:35 PM, Gord Tanner wrote: > > > Since we had cleaned up the lint errors last week I have noticed a few > > more sneak in (including ones I have added :( ). > > > > Could we somehow enforce / ensure we have people running the jake > > command to check lint before commiting / merging? > > > > Ideally I would like to keep the number of lint errors to 0 to prevent > > the massive debt we have incurred so far. > > > -- Patrick Mueller http://muellerware.org --14dae9ccdd4e9b8f2a04cba7aa26--