Add explanation why we should return early for GitHub importer

bfc529bb · Douglas Barbosa Alexandre · e198eea9 · bfc529bb
Commit bfc529bb authored Aug 07, 2017 by Douglas Barbosa Alexandre
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

app/services/projects/import_service.rb app/services/projects/import_service.rb +2 -0

No files found.
--- a/app/services/projects/import_service.rb
+++ b/app/services/projects/import_service.rb
@@ -34,6 +34,8 @@ module Projects
    def import_repository
      raise Error, 'Blocked import URL.' if Gitlab::UrlBlocker.blocked_url?(project.import_url)
+      # We should return early for a GitHub import because the new GitHub
+      # importer fetch the project repositories for us.
      return if project.github_import?
      begin