From 45ef2a5508ed4485a0ee15354dcb1e3829fa9a6e Mon Sep 17 00:00:00 2001 From: Josh Callender Date: Fri, 24 Apr 2015 12:00:41 -0700 Subject: add support for CircleCI pull request commenting --- lib/getOptions.js | 1 + 1 file changed, 1 insertion(+) (limited to 'lib/getOptions.js') diff --git a/lib/getOptions.js b/lib/getOptions.js index eab67f4..4289ad9 100644 --- a/lib/getOptions.js +++ b/lib/getOptions.js @@ -35,6 +35,7 @@ var getBaseOptions = function(cb){ if (process.env.CIRCLECI){ options.service_name = 'circleci'; options.service_job_id = process.env.CIRCLE_BUILD_NUM; + options.service_pull_request = process.env.CI_PULL_REQUEST; git_commit = process.env.CIRCLE_SHA1; git_branch = process.env.CIRCLE_BRANCH; } -- cgit v1.2.3 From fba900513bb8653d8d83b7c35053be72d519d5d6 Mon Sep 17 00:00:00 2001 From: Josh Callender Date: Fri, 24 Apr 2015 12:57:16 -0700 Subject: CI_PULL_REQUEST is just the url for the entire PR, have to split it and grab the number there. CIRCLE_PR_NUMBER is only defined for forks. --- lib/getOptions.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'lib/getOptions.js') diff --git a/lib/getOptions.js b/lib/getOptions.js index 4289ad9..e729e53 100644 --- a/lib/getOptions.js +++ b/lib/getOptions.js @@ -35,7 +35,11 @@ var getBaseOptions = function(cb){ if (process.env.CIRCLECI){ options.service_name = 'circleci'; options.service_job_id = process.env.CIRCLE_BUILD_NUM; - options.service_pull_request = process.env.CI_PULL_REQUEST; + + if (process.env.CI_PULL_REQUEST) { + var pr = process.env.CI_PULL_REQUEST.split('/pull/'); + options.service_pull_request = pr[1]; + } git_commit = process.env.CIRCLE_SHA1; git_branch = process.env.CIRCLE_BRANCH; } -- cgit v1.2.3