copyright check pre-commit hook

Ben Finney ben+bazaar at benfinney.id.au
Thu Feb 11 00:13:54 GMT 2010


John Arbash Meinel <john at arbash-meinel.com> writes:

> Right. It is harder to review code that has a lot of lines 'changed'
> just because of trailing whitespace. In theory that was why we wanted
> to push to clean up the codebase and then force trailing whitespace to
> be removed (in pqm). Robert found it frustrating to have things
> rejected by pqm just because his editor left trailing whitespace that
> he didn't see, so that got pushed back.

The solution, of course, is to discover the trailing whitespace long
*before* it gets to pqm. Preferably, before committing at all.

-- 
 \          Eccles: “I'll get [the job] too, you'll see. I'm wearing a |
  `\        Cambridge tie.”  Greenslade: “What were you doing there?”  |
_o__)   Eccles: “Buying a tie.” —The Goon Show, _The Greenslade Story_ |
Ben Finney




More information about the bazaar mailing list