From 4cb73a3b31cd205ef198edbfcba7eef1f1add40f Mon Sep 17 00:00:00 2001 From: ABaldwinHunter Date: Mon, 14 Sep 2015 22:32:35 -0400 Subject: [PATCH] Don't remove details_link for errored PRs Now that we have a snapshot events page, don't remove details url for error. Change in finalizer: https://github.com/codeclimate/finalizer/commit/d63a1d1d29d112901a5bf2352939ae545259a57b --- lib/cc/services/github_pull_requests.rb | 3 --- 1 file changed, 3 deletions(-) diff --git a/lib/cc/services/github_pull_requests.rb b/lib/cc/services/github_pull_requests.rb index 85f5b1b..ed6b109 100644 --- a/lib/cc/services/github_pull_requests.rb +++ b/lib/cc/services/github_pull_requests.rb @@ -109,9 +109,6 @@ def update_status(state, description) target_url: @payload["details_url"], context: "codeclimate" } - if state == "error" - params.delete(:target_url) - end @response = service_post(status_url, params.to_json) end end