It looks like the reason for the double-posting is that tricium-dev and tricium-prod are both posting to Gerrit; this is happening in the tools repo because the tools repo entry in tricium-dev.cfg doesn't have disable_reporting: true.


The suggestion to check spelling only in comments is worth considering.

However, I argue there's also reason to check non-comments, since we might also want to catch possible misspellings in strings and possibly also variable names; also, it only flags things that *look like* misspellings, so it doesn't flag "preProc" or "ctx"; but it flags "pres" because it looks like a possible misspelling.

I see the logic, but particularly in languages like Go the convention is to use highly abbreviated names, which would be caught by the spell checker like this. It just seems like unnecessary noise that will annoy developers, not help them.

Yeah, we definitely don't want to make unnecessary noise that will annoy developers. Filed separate  bug 867538  for that issue.