[#359] Return repo store type when getting a PR repository and number

UffizziCloud · Feb 2, 2023 · 49921a3 · 49921a3
1 parent 151cc25
commit 49921a3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/core/app/services/uffizzi_core/deployment_service.rb b/core/app/services/uffizzi_core/deployment_service.rb
@@ -99,7 +99,7 @@ def build_subdomain(deployment)
     end
 
     def build_pull_request_subdomain(deployment)
-      repo_name, pull_request_number = pull_request_data(deployment)
+      _, repo_name, pull_request_number = pull_request_data(deployment)
       raise UffizziCore::Deployment::LabelsNotFoundError if repo_name.nil? || pull_request_number.nil?
 
       formatted_repo_name = format_url_safe(repo_name.split('/').last.downcase)
@@ -307,13 +307,13 @@ def pull_request_data(deployment)
     def github_pull_request_data(deployment)
       github_data = deployment.metadata.dig('labels', 'github')
 
-      [github_data['repository'], github_data.dig('event', 'number')]
+      [:github, github_data['repository'], github_data.dig('event', 'number')]
     end
 
     def gitlab_merge_request_data(deployment)
       gitlab_data = deployment.metadata.dig('labels', 'gitlab')
 
-      [gitlab_data['repo'], gitlab_data.dig('merge_request', 'number')]
+      [:gitlab, gitlab_data['repo'], gitlab_data.dig('merge_request', 'number')]
     end
 
     def format_url_safe(name)