Fix base branch SHA usage with GitHub Actions (#9257)

daniel-mohedano · web-flow · commit bf6c156e8c1e · 2025-07-30T10:00:52.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
@@ -125,13 +125,15 @@ static PullRequestInfo buildPullRequestInfo(
     }
 
     // complete with CI vars if user didn't provide all information
-    PullRequestInfo ciInfo = PullRequestInfo.merge(userInfo, ciProviderInfo.buildPullRequestInfo());
+    PullRequestInfo ciInfo =
+        PullRequestInfo.coalesce(userInfo, ciProviderInfo.buildPullRequestInfo());
     String headSha = ciInfo.getHeadCommit().getSha();
     if (Strings.isNotBlank(headSha)) {
       // 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.coalesce(
+            ciInfo, new PullRequestInfo(null, null, null, commitInfo, null));
       } catch (Exception ignored) {
       }
     }
@@ -145,6 +147,7 @@ private static PullRequestInfo buildUserPullRequestInfo(
         new PullRequestInfo(
             config.getGitPullRequestBaseBranch(),
             config.getGitPullRequestBaseBranchSha(),
+            null,
             new CommitInfo(config.getGitCommitHeadSha()),
             null);
 
@@ -164,10 +167,11 @@ private static PullRequestInfo buildUserPullRequestInfo(
         new PullRequestInfo(
             null,
             mergeBase,
+            null,
             new CommitInfo(environment.get(Constants.DDCI_PULL_REQUEST_SOURCE_SHA)),
             null);
 
-    return PullRequestInfo.merge(userInfo, ddCiInfo);
+    return PullRequestInfo.coalesce(userInfo, ddCiInfo);
   }
 
   private static String getRepoRoot(CIInfo ciInfo, GitClient.Factory gitClientFactory) {
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
@@ -91,6 +91,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
@@ -86,6 +86,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
@@ -42,6 +42,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)
@@ -135,13 +136,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
@@ -62,7 +62,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
@@ -89,6 +89,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
@@ -108,7 +108,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;
 
@@ -121,7 +121,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");
@@ -130,11 +130,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,38 +51,36 @@ 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);
   }
 
   /**
-   * Merges info by completing the empty information fields with the fallback's
+   * Combine infos by completing the empty information fields in {@code first} with {@code second}'s
    *
-   * @param info Base PR info
-   * @param fallback Fallback PR info
-   * @return Completed PR info
+   * @param first Base PR info
+   * @param second Fallback PR info
+   * @return Combined PR info
    */
-  public static PullRequestInfo merge(PullRequestInfo info, PullRequestInfo fallback) {
+  public static PullRequestInfo coalesce(
+      final PullRequestInfo first, final PullRequestInfo second) {
     return new PullRequestInfo(
-        Strings.isNotBlank(info.pullRequestBaseBranch)
-            ? info.pullRequestBaseBranch
-            : fallback.pullRequestBaseBranch,
-        Strings.isNotBlank(info.pullRequestBaseBranchSha)
-            ? info.pullRequestBaseBranchSha
-            : fallback.pullRequestBaseBranchSha,
-        CommitInfo.merge(info.headCommit, fallback.headCommit),
-        Strings.isNotBlank(info.pullRequestNumber)
-            ? info.pullRequestNumber
-            : fallback.pullRequestNumber);
+        Strings.coalesce(first.baseBranch, second.baseBranch),
+        Strings.coalesce(first.baseBranchSha, second.baseBranchSha),
+        Strings.coalesce(first.baseBranchHeadSha, second.baseBranchHeadSha),
+        CommitInfo.coalesce(first.headCommit, second.headCommit),
+        Strings.coalesce(first.pullRequestNumber, second.pullRequestNumber));
   }
 
   @Override
@@ -87,26 +92,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
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/api/git/CommitInfo.java b/internal-api/src/main/java/datadog/trace/api/git/CommitInfo.java
diff --git a/internal-api/src/main/java/datadog/trace/api/git/PersonInfo.java b/internal-api/src/main/java/datadog/trace/api/git/PersonInfo.java
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
diff --git a/internal-api/src/main/java/datadog/trace/util/Strings.java b/internal-api/src/main/java/datadog/trace/util/Strings.java
diff --git a/internal-api/src/test/groovy/datadog/trace/api/civisibility/git/CommitInfoTest.groovy b/internal-api/src/test/groovy/datadog/trace/api/civisibility/git/CommitInfoTest.groovy
diff --git a/internal-api/src/test/groovy/datadog/trace/api/civisibility/git/PersonInfoTest.groovy b/internal-api/src/test/groovy/datadog/trace/api/civisibility/git/PersonInfoTest.groovy
diff --git a/internal-api/src/test/groovy/datadog/trace/util/StringsTest.groovy b/internal-api/src/test/groovy/datadog/trace/util/StringsTest.groovy

Original file line number	Diff line number	Diff line change
`@@ -91,6 +91,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`91`	`91`	`return new PullRequestInfo(`
`92`	`92`	`normalizeBranch(environment.get(AZURE_PR_TARGET_BRANCH)),`
`93`	`93`	`null,`
	`94`	`+ null,`
`94`	`95`	`CommitInfo.NOOP,`
`95`	`96`	`environment.get(AZURE_PR_NUMBER));`
`96`	`97`	`}`
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
`@@ -86,6 +86,7 @@ public PullRequestInfo buildPullRequestInfo() {`
`86`	`86`	`return new PullRequestInfo(`
`87`	`87`	`normalizeBranch(environment.get(BUILDKITE_PULL_REQUEST_BASE_BRANCH)),`
`88`	`88`	`null,`
	`89`	`+ null,`
`89`	`90`	`CommitInfo.NOOP,`
`90`	`91`	`environment.get(BUILDKITE_PULL_REQUEST_NUMBER));`
`91`	`92`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,8 @@ public CIInfo buildCIInfo() {`
`62`	`62`	`@Nonnull`
`63`	`63`	`@Override`
`64`	`64`	`public PullRequestInfo buildPullRequestInfo() {`
`65`		`- return new PullRequestInfo(null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));`
	`65`	`+ return new PullRequestInfo(`
	`66`	`+ null, null, null, CommitInfo.NOOP, environment.get(CIRCLECI_PR_NUMBER));`
`66`	`67`	`}`
`67`	`68`
`68`	`69`	`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`	`}`