David Pordomingo

Results 65 comments of David Pordomingo

Using dep it can be locked. I'll pull request it tomorrow morning after the Empathy Session :dagger:

This issue was dropped from OKRs, but can be considered as a feature-idea here

to post regular messages, the way that @smacker suggested should be the ok. But for other purposes Lookout could increase its api; example: `isMergeable`, as part of the Analyzer `EventResponse`,...

Travis is failing at `Generated code` stage https://travis-ci.com/src-d/lookout/jobs/187599618 because `vendor` is currently hardcoded at 9a4e7e2 (because it has not been yet merged on `go-git` nor `go-git-fixtures`)

It seems that DataService no longer crash if there are changes in a tree that is a git sub-module; @bzz suggested that `DiffTreeScanner` should not even detect these as a...

There is already (for workers and watcher): - `/health/liveness`, returns `200` if it was started - `/health/readiness`, returns `200` if all startup instructions succeded And for `dummy`: - `/health/liveness`, returns...

I agree with you @smacker about criticalness of DB and bblfsh. would you create two new endpoints for them? or should we adapt /health/readiness to also check the DB and...

I think GitHub API is returning an unhandled `Path is invalid` error due to https://github.com/google/go-github/issues/540

I like the proposal; @carlosms should we consider it, or do you have any other idea?

**From your first answer:** >_same comment posted twice on different pushes is annoying, but only if it was resolved or addressed on the first push; if it was not, then...