repo-updater: use proper nil check for info.CloneTime (!3643) · Merge requests · Warren Gifford / sourcegraph

Warren Gifford requested to merge fix-clone-time-repo into master Apr 26, 2019

Created by: ggilmore

https://github.com/sourcegraph/sourcegraph/commit/68b1d86ccc16b758b3595c0a54deb8a2d010fa9a introduced a regression caused the info.CloneTime pointer to be dereferenced before checking to see if was nil on the following line. This caused panics in production.

This PR swaps the order of the nil check and the pointer deference - which should alleviate this issue.

repo-updater: use proper nil check for info.CloneTime