Merge pull request #1539 from miguno/issue-1538
Fix: correctly parse git remote origin urls for calling Octokit
This commit is contained in:
@@ -43,7 +43,7 @@ class GithubStatusChecks
|
|||||||
attr_reader :remote_url, :git_sha, :github_client, :combined_status
|
attr_reader :remote_url, :git_sha, :github_client, :combined_status
|
||||||
|
|
||||||
def initialize
|
def initialize
|
||||||
@remote_url = `git config --get remote.origin.url`.strip.delete_prefix("https://github.com/")
|
@remote_url = github_repo_from_remote_url
|
||||||
@git_sha = `git rev-parse HEAD`.strip
|
@git_sha = `git rev-parse HEAD`.strip
|
||||||
@github_client = Octokit::Client.new(access_token: ENV["GITHUB_TOKEN"])
|
@github_client = Octokit::Client.new(access_token: ENV["GITHUB_TOKEN"])
|
||||||
refresh!
|
refresh!
|
||||||
@@ -77,6 +77,18 @@ class GithubStatusChecks
|
|||||||
"Build not started..."
|
"Build not started..."
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
private
|
||||||
|
def github_repo_from_remote_url
|
||||||
|
url = `git config --get remote.origin.url`.strip.delete_suffix(".git")
|
||||||
|
if url.start_with?("https://github.com/")
|
||||||
|
url.delete_prefix("https://github.com/")
|
||||||
|
elsif url.start_with?("git@github.com:")
|
||||||
|
url.delete_prefix("git@github.com:")
|
||||||
|
else
|
||||||
|
url
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user