We have been using houndci for a bit and it has been working fine for us. Though the fact that it comments on auto-generated files and there is no way to get it to ignore those is very irritating.
Thanks to learning Fridays, I spent some time last Friday and today to build a simple Go program which can monitor your PRs, run them through golint and comment on your PR. Pretty much what hound does, though now we have the ability to ignore files with
- An extension
- In a folder
- A particular file
I also noticed that on multiple iteration in a PR hound comments on the same line again even though it wasn’t changed in that particular iteration. This shouldn’t happen with Dgraph lint because we only look the changed lines in an iteration.
I also plan to add support for other linters like go vet and errcheck for now. I will run this in debug (comments are not published to Github but just printed to console) mode for a week, if it works fine then we can replace hound with this.
I think the only disadvantage of this is that we and anyone who uses it has to host it themselves whereas hound is easier to setup because it automates the access token generation process using OAuth2 and persists those. Maybe I can try that later.
Looking forward to feedback from you guys. https://github.com/dgraph-io/lint