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 <gtan...@gmail.com> 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. >
