fix: use base SHA value in GHA as base branch head sha

daniel-mohedano · daniel-mohedano · commit cff717097780 · 2025-07-28T16:02:48.000+02:00
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/CiVisibilityRepoServices.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/CiVisibilityRepoServices.java
@@ -131,7 +131,8 @@ static PullRequestInfo buildPullRequestInfo(
       // if head sha present try to populate author, committer and message info through git client
       try {
         CommitInfo commitInfo = gitClient.getCommitInfo(headSha, true);
-        return PullRequestInfo.merge(ciInfo, new PullRequestInfo(null, null, commitInfo, null));
+        return PullRequestInfo.merge(
+            ciInfo, new PullRequestInfo(null, null, null, commitInfo, null));
       } catch (Exception ignored) {
       }
     }
@@ -145,6 +146,7 @@ private static PullRequestInfo buildUserPullRequestInfo(
         new PullRequestInfo(
             config.getGitPullRequestBaseBranch(),
             config.getGitPullRequestBaseBranchSha(),
+            null,
             new CommitInfo(config.getGitCommitHeadSha()),
             null);
 
@@ -164,6 +166,7 @@ private static PullRequestInfo buildUserPullRequestInfo(
         new PullRequestInfo(
             null,
             mergeBase,
+            null,
             new CommitInfo(environment.get(Constants.DDCI_PULL_REQUEST_SOURCE_SHA)),
             null);
 
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AppVeyorInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AppVeyorInfo.java
@@ -91,6 +91,7 @@ public PullRequestInfo buildPullRequestInfo() {
       return new PullRequestInfo(
           normalizeBranch(environment.get(APPVEYOR_REPO_BRANCH)),
           null,
+          null,
           new CommitInfo(environment.get(APPVEYOR_PR_HEAD_COMMIT)),
           environment.get(APPVEYOR_PR_NUMBER));
     } else {
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AzurePipelinesInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/AzurePipelinesInfo.java
@@ -90,6 +90,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(AZURE_PR_TARGET_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(AZURE_PR_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitBucketInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitBucketInfo.java
@@ -79,6 +79,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(BITBUCKET_PR_DESTINATION_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(BITBUCKET_PR_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitriseInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BitriseInfo.java
@@ -76,6 +76,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(BITRISE_GIT_BRANCH_DEST)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(BITRISE_PR_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuddyInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuddyInfo.java
@@ -67,6 +67,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(BUDDY_RUN_PR_BASE_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(BUDDY_RUN_PR_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuildkiteInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/BuildkiteInfo.java
@@ -85,6 +85,7 @@ public PullRequestInfo buildPullRequestInfo() {
       return new PullRequestInfo(
           normalizeBranch(environment.get(BUILDKITE_PULL_REQUEST_BASE_BRANCH)),
           null,
+          null,
           CommitInfo.NOOP,
           environment.get(BUILDKITE_PULL_REQUEST_NUMBER));
     }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CITagsProvider.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CITagsProvider.java
@@ -41,6 +41,7 @@ public Map<String, String> getCiTags(CIInfo ciInfo, PullRequestInfo pullRequestI
         .withAdditionalTags(ciInfo.getAdditionalTags())
         .withPullRequestBaseBranch(pullRequestInfo)
         .withPullRequestBaseBranchSha(pullRequestInfo)
+        .withPullRequestBaseBranchHeadSha(pullRequestInfo)
         .withGitCommitHeadSha(pullRequestInfo)
         .withGitCommitHeadAuthorName(pullRequestInfo)
         .withGitCommitHeadAuthorEmail(pullRequestInfo)
@@ -130,13 +131,16 @@ public CITagsBuilder withAdditionalTags(final Map<String, String> additionalTags
     }
 
     public CITagsBuilder withPullRequestBaseBranch(final PullRequestInfo pullRequestInfo) {
-      return putTagValue(
-          Tags.GIT_PULL_REQUEST_BASE_BRANCH, pullRequestInfo.getPullRequestBaseBranch());
+      return putTagValue(Tags.GIT_PULL_REQUEST_BASE_BRANCH, pullRequestInfo.getBaseBranch());
     }
 
     public CITagsBuilder withPullRequestBaseBranchSha(final PullRequestInfo pullRequestInfo) {
+      return putTagValue(Tags.GIT_PULL_REQUEST_BASE_BRANCH_SHA, pullRequestInfo.getBaseBranchSha());
+    }
+
+    public CITagsBuilder withPullRequestBaseBranchHeadSha(final PullRequestInfo pullRequestInfo) {
       return putTagValue(
-          Tags.GIT_PULL_REQUEST_BASE_BRANCH_SHA, pullRequestInfo.getPullRequestBaseBranchSha());
+          Tags.GIT_PULL_REQUEST_BASE_BRANCH_HEAD_SHA, pullRequestInfo.getBaseBranchHeadSha());
     }
 
     public CITagsBuilder withGitCommitHeadSha(final PullRequestInfo pullRequestInfo) {
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CircleCIInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CircleCIInfo.java
@@ -61,7 +61,8 @@ public CIInfo buildCIInfo() {
   @Nonnull
   @Override
   public PullRequestInfo buildPullRequestInfo() {
-    return new PullRequestInfo(null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));
+    return new PullRequestInfo(
+        null, null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));
   }
 
   private String buildPipelineUrl(final String pipelineId) {
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CodefreshInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/CodefreshInfo.java
@@ -51,6 +51,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(CF_PULL_REQUEST_TARGET_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(CF_PULL_REQUEST_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/DroneInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/DroneInfo.java
@@ -68,6 +68,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(DRONE_PULL_REQUEST_TARGET_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(DRONE_PULL_REQUEST_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/GitLabInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/GitLabInfo.java
@@ -88,6 +88,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(GITLAB_PULL_REQUEST_BASE_BRANCH)),
         null,
+        null,
         new CommitInfo(environment.get(GITLAB_PULL_REQUEST_COMMIT_HEAD_SHA)),
         environment.get(GITLAB_PULL_REQUEST_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/GithubActionsInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/GithubActionsInfo.java
@@ -107,7 +107,7 @@ public PullRequestInfo buildPullRequestInfo() {
           moshi.adapter(Types.newParameterizedType(Map.class, String.class, Object.class));
       Map<String, Object> eventJson = mapJsonAdapter.fromJson(event);
 
-      String baseSha = null;
+      String baseBranchHeadSha = null;
       String headSha = null;
       String prNumber = null;
 
@@ -120,7 +120,7 @@ public PullRequestInfo buildPullRequestInfo() {
 
         Map<String, Object> base = (Map<String, Object>) pullRequest.get("base");
         if (base != null) {
-          baseSha = (String) base.get("sha");
+          baseBranchHeadSha = (String) base.get("sha");
         }
 
         Double number = (Double) pullRequest.get("number");
@@ -129,11 +129,12 @@ public PullRequestInfo buildPullRequestInfo() {
         }
       }
 
-      return new PullRequestInfo(baseRef, baseSha, new CommitInfo(headSha), prNumber);
+      return new PullRequestInfo(
+          baseRef, null, baseBranchHeadSha, new CommitInfo(headSha), prNumber);
 
     } catch (Exception e) {
       LOGGER.warn("Error while parsing GitHub event", e);
-      return new PullRequestInfo(baseRef, null, CommitInfo.NOOP, null);
+      return new PullRequestInfo(baseRef, null, null, CommitInfo.NOOP, null);
     }
   }
 
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/JenkinsInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/JenkinsInfo.java
@@ -77,6 +77,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(JENKINS_PR_BASE_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(JENKINS_PR_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/PullRequestInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/PullRequestInfo.java
@@ -8,30 +8,37 @@
 public class PullRequestInfo {
 
   public static final PullRequestInfo EMPTY =
-      new PullRequestInfo(null, null, CommitInfo.NOOP, null);
+      new PullRequestInfo(null, null, null, CommitInfo.NOOP, null);
 
-  private final String pullRequestBaseBranch;
-  private final String pullRequestBaseBranchSha;
+  private final String baseBranch;
+  private final String baseBranchSha;
+  private final String baseBranchHeadSha;
   @Nonnull private final CommitInfo headCommit;
   private final String pullRequestNumber;
 
   public PullRequestInfo(
-      String pullRequestBaseBranch,
-      String pullRequestBaseBranchSha,
+      String baseBranch,
+      String baseBranchSha,
+      String baseBranchHeadSha,
       @Nonnull CommitInfo headCommit,
       String pullRequestNumber) {
-    this.pullRequestBaseBranch = pullRequestBaseBranch;
-    this.pullRequestBaseBranchSha = pullRequestBaseBranchSha;
+    this.baseBranch = baseBranch;
+    this.baseBranchSha = baseBranchSha;
+    this.baseBranchHeadSha = baseBranchHeadSha;
     this.headCommit = headCommit;
     this.pullRequestNumber = pullRequestNumber;
   }
 
-  public String getPullRequestBaseBranch() {
-    return pullRequestBaseBranch;
+  public String getBaseBranch() {
+    return baseBranch;
   }
 
-  public String getPullRequestBaseBranchSha() {
-    return pullRequestBaseBranchSha;
+  public String getBaseBranchSha() {
+    return baseBranchSha;
+  }
+
+  public String getBaseBranchHeadSha() {
+    return baseBranchHeadSha;
   }
 
   @Nonnull
@@ -44,15 +51,17 @@ public String getPullRequestNumber() {
   }
 
   public boolean isEmpty() {
-    return Strings.isBlank(pullRequestBaseBranch)
-        && Strings.isBlank(pullRequestBaseBranchSha)
+    return Strings.isBlank(baseBranch)
+        && Strings.isBlank(baseBranchSha)
+        && Strings.isBlank(baseBranchHeadSha)
         && headCommit.isEmpty()
         && Strings.isBlank(pullRequestNumber);
   }
 
   public boolean isComplete() {
-    return Strings.isNotBlank(pullRequestBaseBranch)
-        && Strings.isNotBlank(pullRequestBaseBranchSha)
+    return Strings.isNotBlank(baseBranch)
+        && Strings.isNotBlank(baseBranchSha)
+        && Strings.isNotBlank(baseBranchHeadSha)
         && headCommit.isComplete()
         && Strings.isNotBlank(pullRequestNumber);
   }
@@ -66,12 +75,11 @@ public boolean isComplete() {
    */
   public static PullRequestInfo merge(PullRequestInfo info, PullRequestInfo fallback) {
     return new PullRequestInfo(
-        Strings.isNotBlank(info.pullRequestBaseBranch)
-            ? info.pullRequestBaseBranch
-            : fallback.pullRequestBaseBranch,
-        Strings.isNotBlank(info.pullRequestBaseBranchSha)
-            ? info.pullRequestBaseBranchSha
-            : fallback.pullRequestBaseBranchSha,
+        Strings.isNotBlank(info.baseBranch) ? info.baseBranch : fallback.baseBranch,
+        Strings.isNotBlank(info.baseBranchSha) ? info.baseBranchSha : fallback.baseBranchSha,
+        Strings.isNotBlank(info.baseBranchHeadSha)
+            ? info.baseBranchHeadSha
+            : fallback.baseBranchHeadSha,
         CommitInfo.merge(info.headCommit, fallback.headCommit),
         Strings.isNotBlank(info.pullRequestNumber)
             ? info.pullRequestNumber
@@ -87,26 +95,29 @@ public boolean equals(Object o) {
       return false;
     }
     PullRequestInfo that = (PullRequestInfo) o;
-    return Objects.equals(pullRequestBaseBranch, that.pullRequestBaseBranch)
-        && Objects.equals(pullRequestBaseBranchSha, that.pullRequestBaseBranchSha)
+    return Objects.equals(baseBranch, that.baseBranch)
+        && Objects.equals(baseBranchSha, that.baseBranchSha)
+        && Objects.equals(baseBranchHeadSha, that.baseBranchHeadSha)
         && Objects.equals(headCommit, that.headCommit)
         && Objects.equals(pullRequestNumber, that.pullRequestNumber);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(
-        pullRequestBaseBranch, pullRequestBaseBranchSha, headCommit, pullRequestNumber);
+    return Objects.hash(baseBranch, baseBranchSha, headCommit, pullRequestNumber);
   }
 
   @Override
   public String toString() {
     return "PR{"
         + "baseBranch='"
-        + pullRequestBaseBranch
+        + baseBranch
         + '\''
         + ", baseSHA='"
-        + pullRequestBaseBranchSha
+        + baseBranchSha
+        + '\''
+        + ", baseHeadSHA='"
+        + baseBranchHeadSha
         + '\''
         + ", headCommit='"
         + headCommit
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/TeamcityInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/TeamcityInfo.java
@@ -43,6 +43,7 @@ public PullRequestInfo buildPullRequestInfo() {
     return new PullRequestInfo(
         normalizeBranch(environment.get(TEAMCITY_PULL_REQUEST_TARGET_BRANCH)),
         null,
+        null,
         CommitInfo.NOOP,
         environment.get(TEAMCITY_PULL_REQUEST_NUMBER));
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/TravisInfo.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/ci/TravisInfo.java
@@ -71,6 +71,7 @@ public PullRequestInfo buildPullRequestInfo() {
       return new PullRequestInfo(
           normalizeBranch(environment.get(TRAVIS_GIT_BRANCH)),
           null,
+          null,
           new CommitInfo(environment.get(TRAVIS_PR_HEAD_SHA)),
           environment.get(TRAVIS_PR_NUMBER));
     }
diff --git a/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/config/ExecutionSettingsFactoryImpl.java b/dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/config/ExecutionSettingsFactoryImpl.java
@@ -423,10 +423,16 @@ private Diff getPullRequestDiff(boolean impactedTestsDetectionEnabled, String de
         // ensure repo is not shallow before attempting to get git diff
         gitRepoUnshallow.unshallow();
 
-        String baseCommitSha = pullRequestInfo.getPullRequestBaseBranchSha();
+        String baseCommitSha = pullRequestInfo.getBaseBranchSha();
+        if (baseCommitSha == null && pullRequestInfo.getBaseBranchHeadSha() != null) {
+          baseCommitSha =
+              gitClient.getMergeBase(
+                  pullRequestInfo.getBaseBranchHeadSha(), pullRequestInfo.getHeadCommit().getSha());
+        }
+
         if (baseCommitSha == null) {
           baseCommitSha =
-              gitClient.getBaseCommitSha(pullRequestInfo.getPullRequestBaseBranch(), defaultBranch);
+              gitClient.getBaseCommitSha(pullRequestInfo.getBaseBranch(), defaultBranch);
         }
 
         Diff diff = gitClient.getGitDiff(baseCommitSha, pullRequestInfo.getHeadCommit().getSha());
diff --git a/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/CiVisibilityRepoServicesTest.groovy b/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/CiVisibilityRepoServicesTest.groovy
@@ -36,7 +36,7 @@ class CiVisibilityRepoServicesTest extends Specification {
     setup:
     def expectedInfo = new PullRequestInfo(
       "master",
-      "baseSha",
+      "baseSha", null,
       new CommitInfo(
       "sourceSha",
       new PersonInfo("john", "john@doe.com", "never"),
@@ -47,18 +47,18 @@ class CiVisibilityRepoServicesTest extends Specification {
       )
 
     def config = Stub(Config)
-    config.getGitPullRequestBaseBranch() >> expectedInfo.getPullRequestBaseBranch()
+    config.getGitPullRequestBaseBranch() >> expectedInfo.getBaseBranch()
 
     def environment = Stub(CiEnvironment)
     environment.get(Constants.DDCI_PULL_REQUEST_TARGET_SHA) >> "targetSha"
     environment.get(Constants.DDCI_PULL_REQUEST_SOURCE_SHA) >> expectedInfo.getHeadCommit().getSha()
 
     def repoUnshallow = Stub(GitRepoUnshallow)
     def ciProviderInfo = Stub(CIProviderInfo)
-    ciProviderInfo.buildPullRequestInfo() >> new PullRequestInfo(null, null, CommitInfo.NOOP, expectedInfo.getPullRequestNumber())
+    ciProviderInfo.buildPullRequestInfo() >> new PullRequestInfo(null, null, null, CommitInfo.NOOP, expectedInfo.getPullRequestNumber())
 
     def gitClient = Stub(GitClient)
-    gitClient.getMergeBase("targetSha", "sourceSha") >> expectedInfo.getPullRequestBaseBranchSha()
+    gitClient.getMergeBase("targetSha", "sourceSha") >> expectedInfo.getBaseBranchSha()
     gitClient.getCommitInfo("sourceSha", true) >> expectedInfo.getHeadCommit()
 
     expect:
diff --git a/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/ci/GithubActionsInfoTest.groovy b/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/ci/GithubActionsInfoTest.groovy
@@ -40,8 +40,9 @@ class GithubActionsInfoTest extends CITagsProviderTest {
     def pullRequestInfo = new GithubActionsInfo(new CiEnvironmentImpl(System.getenv())).buildPullRequestInfo()
 
     then:
-    pullRequestInfo.getPullRequestBaseBranch() == "base-ref"
-    pullRequestInfo.getPullRequestBaseBranchSha() == "52e0974c74d41160a03d59ddc73bb9f5adab054b"
+    pullRequestInfo.getBaseBranch() == "base-ref"
+    pullRequestInfo.getBaseBranchSha() == null
+    pullRequestInfo.getBaseBranchHeadSha() == "52e0974c74d41160a03d59ddc73bb9f5adab054b"
     pullRequestInfo.getHeadCommit().getSha() == "df289512a51123083a8e6931dd6f57bb3883d4c4"
     pullRequestInfo.getPullRequestNumber() == "1"
   }
diff --git a/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/ci/PullRequestInfoTest.groovy b/dd-java-agent/agent-ci-visibility/src/test/groovy/datadog/trace/civisibility/ci/PullRequestInfoTest.groovy
diff --git a/internal-api/src/main/java/datadog/trace/bootstrap/instrumentation/api/Tags.java b/internal-api/src/main/java/datadog/trace/bootstrap/instrumentation/api/Tags.java

Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`90`	`90`	`return new PullRequestInfo(`
`91`	`91`	`normalizeBranch(environment.get(AZURE_PR_TARGET_BRANCH)),`
`92`	`92`	`null,`
	`93`	`+ null,`
`93`	`94`	`CommitInfo.NOOP,`
`94`	`95`	`environment.get(AZURE_PR_NUMBER));`
`95`	`96`	`}`
Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`79`	`79`	`return new PullRequestInfo(`
`80`	`80`	`normalizeBranch(environment.get(BITBUCKET_PR_DESTINATION_BRANCH)),`
`81`	`81`	`null,`
	`82`	`+ null,`
`82`	`83`	`CommitInfo.NOOP,`
`83`	`84`	`environment.get(BITBUCKET_PR_NUMBER));`
`84`	`85`	`}`
Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`76`	`76`	`return new PullRequestInfo(`
`77`	`77`	`normalizeBranch(environment.get(BITRISE_GIT_BRANCH_DEST)),`
`78`	`78`	`null,`
	`79`	`+ null,`
`79`	`80`	`CommitInfo.NOOP,`
`80`	`81`	`environment.get(BITRISE_PR_NUMBER));`
`81`	`82`	`}`
Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`67`	`67`	`return new PullRequestInfo(`
`68`	`68`	`normalizeBranch(environment.get(BUDDY_RUN_PR_BASE_BRANCH)),`
`69`	`69`	`null,`
	`70`	`+ null,`
`70`	`71`	`CommitInfo.NOOP,`
`71`	`72`	`environment.get(BUDDY_RUN_PR_NUMBER));`
`72`	`73`	`}`
Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`85`	`85`	`return new PullRequestInfo(`
`86`	`86`	`normalizeBranch(environment.get(BUILDKITE_PULL_REQUEST_BASE_BRANCH)),`
`87`	`87`	`null,`
	`88`	`+ null,`
`88`	`89`	`CommitInfo.NOOP,`
`89`	`90`	`environment.get(BUILDKITE_PULL_REQUEST_NUMBER));`
`90`	`91`	`}`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,8 @@ public CIInfo buildCIInfo() {`
`61`	`61`	`@Nonnull`
`62`	`62`	`@Override`
`63`	`63`	`public PullRequestInfo buildPullRequestInfo() {`
`64`		`- return new PullRequestInfo(null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));`
	`64`	`+ return new PullRequestInfo(`
	`65`	`+ null, null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));`
`65`	`66`	`}`
`66`	`67`
`67`	`68`	`private String buildPipelineUrl(final String pipelineId) {`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`51`	`51`	`return new PullRequestInfo(`
`52`	`52`	`normalizeBranch(environment.get(CF_PULL_REQUEST_TARGET_BRANCH)),`
`53`	`53`	`null,`
	`54`	`+ null,`
`54`	`55`	`CommitInfo.NOOP,`
`55`	`56`	`environment.get(CF_PULL_REQUEST_NUMBER));`
`56`	`57`	`}`