Skip to content

Fix adding public repos when no code hosts are configured

Warren Gifford requested to merge addrepo into master

Created by: nicksnyder

fixes https://github.com/sourcegraph/sourcegraph/issues/1288

https://github.com/sourcegraph/sourcegraph/pull/1267 introduced a regression where the default connection list would get initialized to empty and then the first run through the loop we would read an empty config and conclude that nothing had changed. This was wrong because we need to have a default entry.

The simple fix is to move the equality check AFTER we have run the logic that adds the default value.

Merge request reports