Avoid hardcoding repo in fetch-pr

rails · Oct 9, 2024 · 1e21c1d · 1e21c1d
1 parent 82ae82e
commit 1e21c1d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/bin/fetch-pr b/bin/fetch-pr
@@ -13,7 +13,11 @@ def pull_request_number
   ENV.fetch("BUILDKITE_PULL_REQUEST") { raise "Missing BUILDKITE_PULL_REQUEST!" }
 end
 
-uri = URI("https://api.github.com/repos/rails/rails/pulls/#{pull_request_number}")
+def github_repo
+  ENV.fetch("BUILDKITE_REPO")[%r{github\.com[/:](.+?)(?:\.git)?\z}, 1]
+end
+
+uri = URI("https://api.github.com/repos/#{github_repo}/pulls/#{pull_request_number}")
 
 req = Net::HTTP::Get.new(uri)
 req["Authorization"] = "token #{github_token}"

diff --git a/pipelines/rails-ci/initial.yml b/pipelines/rails-ci/initial.yml
@@ -31,6 +31,7 @@ steps:
       (docker run --rm \
         -v "$$PWD":/app:ro -w /app \
         -e GITHUB_PUBLIC_REPO_TOKEN \
+        -e BUILDKITE_REPO \
         -e BUILDKITE_PULL_REQUEST \
         ruby:latest \
         .buildkite/bin/fetch-pr > .buildkite/tmp/.pr-meta.json) || true