simplify "Is Sourcegraph.com?" check (!2406) · Merge requests · Warren Gifford / sourcegraph

Warren Gifford requested to merge simplify-dotcom-check into master Feb 22, 2019

Created by: sqs

I wanted to write a test for Sourcegraph.com and non-Sourcegraph.com behavior for something, so I wanted to mock the result of SourcegraphDotComMode. It is simpler to mock it if we remove the special-case checking of the URL. That special-case is not necessary anyway because Sourcegraph.com is deployed with SOURCEGRAPHDOTCOM_MODE=true (I confirmed this with kubectl -n prod describe deployment/sourcegraph-frontend on the Sourcegraph.com k8s cluster).

simplify "Is Sourcegraph.com?" check

Merge request reports