David Pordomingo

To do the maths it would be needed three extra properties, providing the totals for bytes, lines and empty lines; ie: `bytesCount`, `linesCount`, `emptyLinesCount` otherwise we can not do the...

I also found this issue during my https://github.com/src-d/empathy-sessions/issues/37#issuecomment-470037191 If you log with `--verbose` it is even said that the config will be used but it won't be the truth: ```shell...

imo to ignore the passed config is a misleading behavior; if the stored config differs from the passed one, `srcd` should say so: failing, restarting with the new config, or...

I'll set you as admin this Night. Thanks

It seems that it can not be set an user as a repo admin, so I create a temporary organization, added you as owners, and moved the repo there. This...

Removed, many thanks.

I think that `init` should not collide with other attempts of initializing the daemon.

We already have a queue (RabbitMQ). Considering the different errors described by @smacker, what would be the retry policy?

alternatives from https://github.com/src-d/lookout/pull/447#issuecomment-452388854 [ImageMagick](https://github.com/ImageMagick/ImageMagick) ```shell convert \ docs/assets/lookout-sdk-seq-diagram.svg lookout-sdk-seq-diagram.png ``` @carlosms said at https://github.com/src-d/lookout/pull/447#issuecomment-452657466 >_(this way) they look worse, the text typography or kerning makes it harder to read._ There...

That footer is built as it follows: ```go fmt.Sprintf(providers.github.comment_footer, analyzer.feedback) ``` see docs at https://docs.sourced.tech/lookout/configuration#custom-footer. Each analyzer defines its `feedback` (url) on their own ([examples](https://github.com/src-d/lookout/blob/master/.helm-staging.yml#L36)), so it can be an...