datadog-ci 1.18.0 → 1.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4673e9680749b01863ddde129169fb985f7be2f46ae83da674b856bc70900dc9
-  data.tar.gz: 5e0c4f0a055d7a3017dc2984d39b142cfcf9f4bfb39dca5713a18af0481276f6
+  metadata.gz: b04703fefecfa1e090ce2265ed419e4fa41c170a35c897ab2ac900fe9bca467b
+  data.tar.gz: b94de33937e2116579dcaf36c7a7c4f6040597420b27dff1e0f47641f1966be2
 SHA512:
-  metadata.gz: 58d120417a247f9e6be958efeea4094e6d3f5dc861df5dbe74428f7968e2dc96156b19d5c3db512f70afa15686997e6b38aae39dbeb8d0bb2f4f19fdb8b524b4
-  data.tar.gz: 7d91c8fc4e85bede1f57dc64a151d4ebfb222be88710f958e8a6538ad37e44cedf56c6379fb1476fb472ebf65db57023915100254e180e6087b2c7c0eb7d78ca
+  metadata.gz: 0bce96aa0950aecccac7b7382c731c62d29035b403231fec3ca7f59bb85768e52e0041cfa709a62ed5054e360979776d37e0a501a587d875f3263e82d455ab14
+  data.tar.gz: 6035f0db0a5a7eb9aae0f51813f330acd3abbe43175000513860abf5ad406f0d76d0aaf8f6e09409f00c251816aae84c7c3757eb56a660fdcefe7e9cbd478d7a

data/CHANGELOG.md

@@ -1,5 +1,26 @@
 ## [Unreleased]
 
+## [1.20.0] - 2025-07-03
+
+### Added
+
+* Add Datadog output to RSpec documentation formatter ([#355][])
+* Add Drone CI support and update CI envs handling ([#349][])
+
+### Fixed
+
+* FIX: send git.tag when requesting library settings if git.branch is not available ([#359][])
+
+## [1.19.0] - 2025-06-16
+
+### Changed
+
+* Impacted tests detection works with line level granularity ([#335][])
+
+### Fixed
+
+* Fix stdin close in Command.popen_with_stdin ([#339][])
+
 ## [1.18.0] - 2025-06-06
 
 ### Added
@@ -453,7 +474,9 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 
 - Ruby versions < 2.7 no longer supported ([#8][])
 
-[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.18.0...main
+[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.20.0...main
+[1.20.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.19.0...v1.20.0
+[1.19.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.18.0...v1.19.0
 [1.18.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.17.0...v1.18.0
 [1.17.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.16.0...v1.17.0
 [1.16.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.15.0...v1.16.0
@@ -647,4 +670,9 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 [#321]: https://github.com/DataDog/datadog-ci-rb/issues/321
 [#323]: https://github.com/DataDog/datadog-ci-rb/issues/323
 [#327]: https://github.com/DataDog/datadog-ci-rb/issues/327
-[#329]: https://github.com/DataDog/datadog-ci-rb/issues/329
+[#329]: https://github.com/DataDog/datadog-ci-rb/issues/329
+[#335]: https://github.com/DataDog/datadog-ci-rb/issues/335
+[#339]: https://github.com/DataDog/datadog-ci-rb/issues/339
+[#349]: https://github.com/DataDog/datadog-ci-rb/issues/349
+[#355]: https://github.com/DataDog/datadog-ci-rb/issues/355
+[#359]: https://github.com/DataDog/datadog-ci-rb/issues/359

data/README.md

@@ -9,34 +9,25 @@ Learn more on our [official website](https://docs.datadoghq.com/tests/) and chec
 ## Features
 
 - [Test Visibility](https://docs.datadoghq.com/tests/) - collect metrics and results for your tests
+- [Flaky test management](https://docs.datadoghq.com/tests/flaky_management/) - track, triage, and remediate flaky tests across your organization. Quarantine or disable problematic tests to keep known flakes from breaking builds, and create cases and Jira issues to track work toward fixes.
+- [Auto test retries](https://docs.datadoghq.com/tests/flaky_tests/auto_test_retries/?tab=ruby) - retrying failing tests up to N times to avoid failing your build due to flaky tests
+- [Early flake detection](https://docs.datadoghq.com/tests/flaky_tests/early_flake_detection/?tab=ruby) - Datadog’s test flakiness solution that identifies flakes early by running newly added tests multiple times
 - [Test impact analysis](https://docs.datadoghq.com/tests/test_impact_analysis/) - save time by selectively running only tests affected by code changes
-- [Flaky test management](https://docs.datadoghq.com/tests/flaky_test_management/) - track, alert, search your flaky tests in Datadog UI
-- [Auto test retries](https://docs.datadoghq.com/tests/flaky_test_management/auto_test_retries/?tab=ruby) - retrying failing tests up to N times to avoid failing your build due to flaky tests
-- [Early flake detection](https://docs.datadoghq.com/tests/flaky_test_management/early_flake_detection/?tab=ruby) - Datadog’s test flakiness solution that identifies flakes early by running newly added tests multiple times
-- [Search and manage CI tests](https://docs.datadoghq.com/tests/search/)
+- [Test Health](https://docs.datadoghq.com/tests/test_health) - The Test Health dashboard provides analytics to help teams manage and optimize their testing in CI. This includes sections showing the current impact of test flakiness and how Test Optimization is mitigating these problems.
+- [Inspect your tests' logs in Datadog](https://docs.datadoghq.com/tests/correlate_logs_and_tests)
 - [Enhance developer workflows](https://docs.datadoghq.com/tests/developer_workflows)
 - [Add custom measures to your tests](https://docs.datadoghq.com/tests/guides/add_custom_measures/?tab=ruby)
 - [Browser tests integration with Datadog RUM](https://docs.datadoghq.com/tests/browser_tests)
 
-## Installation
-
-Add to your Gemfile.
+## Setup
 
-```ruby
-group :test do
-  gem "datadog-ci"
-end
-```
+- [Test visibility setup](https://docs.datadoghq.com/tests/setup/ruby/?tab=cloudciprovideragentless)
+- [Test impact analysis setup](https://docs.datadoghq.com/tests/test_impact_analysis/setup/ruby/?tab=cloudciprovideragentless) (test visibility setup is required before setting up test impact analysis)
 
 ## Upgrade from ddtrace v1.x
 
 If you used [test visibility for Ruby](https://docs.datadoghq.com/tests/setup/ruby/) with [ddtrace](https://github.com/datadog/dd-trace-rb) gem, check out our [upgrade guide](/docs/UpgradeGuide.md).
 
-## Setup
-
-- [Test visibility setup](https://docs.datadoghq.com/tests/setup/ruby/?tab=cloudciprovideragentless)
-- [Test impact analysis setup](https://docs.datadoghq.com/tests/test_impact_analysis/setup/ruby/?tab=cloudciprovideragentless) (test visibility setup is required before setting up test impact analysis)
-
 ## Contributing
 
 See [development guide](/docs/DevelopmentGuide.md), [static typing guide](docs/StaticTypingGuide.md) and [contributing guidelines](/CONTRIBUTING.md).

data/lib/datadog/ci/contrib/rspec/configuration/settings.rb

@@ -25,6 +25,12 @@ module Datadog
               end
             end
 
+            option :datadog_formatter_enabled do |o|
+              o.type :bool
+              o.env Ext::ENV_DATADOG_FORMATTER_ENABLED
+              o.default true
+            end
+
             # internal only
             option :dry_run_enabled do |o|
               o.type :bool

data/lib/datadog/ci/contrib/rspec/documentation_formatter.rb

@@ -0,0 +1,126 @@
+module Datadog
+  module CI
+    module Contrib
+      module RSpec
+        # Instrument RSpec::Core::Formatters::DocumentationFormatter
+        module DocumentationFormatter
+          def self.included(base)
+            base.prepend(InstanceMethods)
+          end
+
+          module InstanceMethods
+            def example_passed(notification)
+              super
+
+              dd_example_finished(notification.example)
+            end
+
+            def example_failed(notification)
+              super
+
+              dd_example_finished(notification.example)
+            end
+
+            def example_pending(notification)
+              super
+
+              dd_example_finished(notification.example)
+            end
+
+            def dump_summary(notification)
+              super
+
+              return unless @dd_flaky_tests || @dd_quarantined_tests || @dd_disabled_tests || @dd_skipped_by_tia_tests
+
+              summary = "\nDatadog Test Optimization Summary:\n"
+              summary << "#{@dd_flaky_tests} flaky detected " if @dd_flaky_tests&.positive?
+              summary << "#{@dd_quarantined_tests} quarantined " if @dd_quarantined_tests&.positive?
+              summary << "#{@dd_disabled_tests} disabled " if @dd_disabled_tests&.positive?
+              summary << "#{@dd_skipped_by_tia_tests} skipped by test impact analysis " if @dd_skipped_by_tia_tests&.positive?
+              summary << "\n"
+
+              output.puts summary
+            end
+
+            private
+
+            def dd_example_finished(example)
+              @group_level += 1
+
+              dd_retries_output(example)
+              dd_test_management_output(example)
+              dd_test_impact_analysis_output(example)
+
+              @group_level -= 1
+            end
+
+            def dd_retries_output(example)
+              if !example.metadata[Ext::METADATA_DD_RETRIES] ||
+                  !CI::Ext::Test::RetryReason::DATADOG_RETRY_REASONS.include?(
+                    example.metadata[Ext::METADATA_DD_RETRY_REASON]
+                  )
+                return
+              end
+
+              retries_count = example.metadata[Ext::METADATA_DD_RETRIES]
+
+              output.puts(
+                "#{current_indentation}| Retried #{retries_count} times by #{dd_retry_source(example.metadata[Ext::METADATA_DD_RETRY_REASON])}"
+              )
+              results = example.metadata[Ext::METADATA_DD_RETRY_RESULTS]
+              results_output = results.map { |status, count| "#{count} / #{retries_count} #{status}" }.join(", ")
+              output.puts(
+                "#{current_indentation}| Results were: #{results_output}"
+              )
+
+              if results[CI::Ext::Test::Status::FAIL] > 0 && results[CI::Ext::Test::Status::PASS] > 0
+                output.puts(
+                  "#{current_indentation}| Flaky test detected"
+                )
+
+                @dd_flaky_tests ||= 0
+                @dd_flaky_tests += 1
+              end
+            end
+
+            def dd_retry_source(reason)
+              case reason
+              when CI::Ext::Test::RetryReason::RETRY_FAILED
+                "Datadog Auto Test Retries"
+              when CI::Ext::Test::RetryReason::RETRY_DETECT_FLAKY
+                "Datadog Early Flake Detection"
+              when CI::Ext::Test::RetryReason::RETRY_FLAKY_FIXED
+                "Datadog Flaky Test Management"
+              else
+                "Datadog Test Optimization"
+              end
+            end
+
+            def dd_test_management_output(example)
+              if example.metadata[Ext::METADATA_DD_QUARANTINED]
+                output.puts("#{current_indentation}| Test was quarantined by Datadog Flaky Test Management")
+
+                @dd_quarantined_tests ||= 0
+                @dd_quarantined_tests += 1
+              end
+
+              if example.metadata[Ext::METADATA_DD_DISABLED]
+                output.puts("#{current_indentation}| Test was disabled by Datadog Flaky Test Management")
+
+                @dd_disabled_tests ||= 0
+                @dd_disabled_tests += 1
+              end
+            end
+
+            def dd_test_impact_analysis_output(example)
+              if example.metadata[Ext::METADATA_DD_SKIPPED_BY_ITR]
+                @dd_skipped_by_tia_tests ||= 0
+                @dd_skipped_by_tia_tests += 1
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/datadog/ci/contrib/rspec/example.rb

@@ -74,6 +74,24 @@ module Datadog
                       exception: execution_result.pending_exception
                     )
                   end
+
+                  if datadog_configuration[:datadog_formatter_enabled]
+                    if test_span&.is_retry?
+                      metadata[Ext::METADATA_DD_RETRIES] ||= 0
+                      metadata[Ext::METADATA_DD_RETRY_RESULTS] ||= Hash.new(0)
+
+                      metadata[Ext::METADATA_DD_RETRIES] += 1
+                      metadata[Ext::METADATA_DD_RETRY_REASON] = test_span&.retry_reason
+                      metadata[Ext::METADATA_DD_RETRY_RESULTS][test_span&.status] += 1
+                    end
+
+                    metadata[Ext::METADATA_DD_QUARANTINED] = true if test_span&.quarantined?
+                    metadata[Ext::METADATA_DD_DISABLED] = true if test_span&.disabled?
+
+                    if test_span&.skipped_by_test_impact_analysis?
+                      metadata[Ext::METADATA_DD_SKIPPED_BY_ITR] = true
+                    end
+                  end
                 end
               end
 

data/lib/datadog/ci/contrib/rspec/ext.rb

@@ -11,6 +11,15 @@ module Datadog
           DEFAULT_SERVICE_NAME = "rspec"
 
           ENV_ENABLED = "DD_TRACE_RSPEC_ENABLED"
+          ENV_DATADOG_FORMATTER_ENABLED = "DD_TRACE_RSPEC_DATADOG_FORMATTER_ENABLED"
+
+          # Metadata keys
+          METADATA_DD_RETRIES = :dd_retries
+          METADATA_DD_RETRY_RESULTS = :dd_retry_results
+          METADATA_DD_RETRY_REASON = :dd_retry_reason
+          METADATA_DD_QUARANTINED = :dd_quarantined
+          METADATA_DD_DISABLED = :dd_disabled
+          METADATA_DD_SKIPPED_BY_ITR = :dd_skipped_by_itr
         end
       end
     end

data/lib/datadog/ci/contrib/rspec/patcher.rb

@@ -5,6 +5,7 @@ require_relative "../patcher"
 require_relative "example"
 require_relative "example_group"
 require_relative "runner"
+require_relative "documentation_formatter"
 
 module Datadog
   module CI
@@ -20,6 +21,7 @@ module Datadog
             ::RSpec::Core::Runner.include(Runner)
             ::RSpec::Core::Example.include(Example)
             ::RSpec::Core::ExampleGroup.include(ExampleGroup)
+            ::RSpec::Core::Formatters::DocumentationFormatter.include(DocumentationFormatter)
           end
         end
       end

data/lib/datadog/ci/ext/environment/configuration_discrepancy_checker.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+
+require_relative "../git"
+require_relative "../telemetry"
+require_relative "../../utils/telemetry"
+
+module Datadog
+  module CI
+    module Ext
+      module Environment
+        class ConfigurationDiscrepancyChecker
+          def initialize(ci_provider_env_tags, local_git_tags, user_provided_tags)
+            @ci_provider_env_tags = ci_provider_env_tags
+            @local_git_tags = local_git_tags
+            @user_provided_tags = user_provided_tags
+          end
+
+          def check_for_discrepancies
+            checks = [
+              {
+                left: normalize_value(@ci_provider_env_tags[Git::TAG_COMMIT_SHA]),
+                right: normalize_value(@local_git_tags[Git::TAG_COMMIT_SHA]),
+                type: "commit_discrepancy",
+                expected: "ci_provider",
+                discrepant: "git_client"
+              },
+              {
+                left: normalize_value(@user_provided_tags[Git::TAG_COMMIT_SHA]),
+                right: normalize_value(@local_git_tags[Git::TAG_COMMIT_SHA]),
+                type: "commit_discrepancy",
+                expected: "user_supplied",
+                discrepant: "git_client"
+              },
+              {
+                left: normalize_value(@user_provided_tags[Git::TAG_COMMIT_SHA]),
+                right: normalize_value(@ci_provider_env_tags[Git::TAG_COMMIT_SHA]),
+                type: "commit_discrepancy",
+                expected: "user_supplied",
+                discrepant: "ci_provider"
+              },
+              {
+                left: normalize_value(@ci_provider_env_tags[Git::TAG_REPOSITORY_URL]),
+                right: normalize_value(@local_git_tags[Git::TAG_REPOSITORY_URL]),
+                type: "repository_discrepancy",
+                expected: "ci_provider",
+                discrepant: "git_client"
+              },
+              {
+                left: normalize_value(@user_provided_tags[Git::TAG_REPOSITORY_URL]),
+                right: normalize_value(@local_git_tags[Git::TAG_REPOSITORY_URL]),
+                type: "repository_discrepancy",
+                expected: "user_supplied",
+                discrepant: "git_client"
+              },
+              {
+                left: normalize_value(@user_provided_tags[Git::TAG_REPOSITORY_URL]),
+                right: normalize_value(@ci_provider_env_tags[Git::TAG_REPOSITORY_URL]),
+                type: "repository_discrepancy",
+                expected: "user_supplied",
+                discrepant: "ci_provider"
+              }
+            ]
+
+            git_info_match = true
+            checks.each do |check|
+              if check[:left] && check[:right] && check[:left] != check[:right]
+                Utils::Telemetry.inc(Ext::Telemetry::METRIC_GIT_COMMIT_SHA_DISCREPANCY, 1, {
+                  type: check[:type].to_s,
+                  expected_provider: check[:expected].to_s,
+                  discrepant_provider: check[:discrepant].to_s
+                })
+                git_info_match = false
+              end
+            end
+
+            Utils::Telemetry.inc(Ext::Telemetry::METRIC_GIT_COMMIT_SHA_MATCH, 1, {
+              matched: git_info_match.to_s
+            })
+          end
+
+          private
+
+          def normalize_value(value)
+            return nil if value.nil? || value == ""
+            value
+          end
+        end
+      end
+    end
+  end
+end

data/lib/datadog/ci/ext/environment/extractor.rb

@@ -49,7 +49,9 @@ module Datadog
               Git::TAG_COMMIT_SHA => @provider.git_commit_sha,
               Git::TAG_PULL_REQUEST_BASE_BRANCH => @provider.git_pull_request_base_branch,
               Git::TAG_PULL_REQUEST_BASE_BRANCH_SHA => @provider.git_pull_request_base_branch_sha,
-              Git::TAG_COMMIT_HEAD_SHA => @provider.git_commit_head_sha
+              Git::TAG_COMMIT_HEAD_SHA => @provider.git_commit_head_sha,
+
+              Environment::TAG_PR_NUMBER => @provider.pr_number
             }
 
             # Normalize Git references and filter sensitive data
@@ -57,13 +59,16 @@ module Datadog
             # Expand ~
             expand_workspace!
 
+            # Convert all tag values to strings
+            @tags.transform_values! { |v| v&.to_s }
+
             # remove empty tags
             @tags.reject! do |_, v|
               # setting type of v here to untyped because steep does not
               # understand `v.nil? || something`
 
               # @type var v: untyped
-              v.nil? || v.strip.empty?
+              v.nil? || v.to_s.strip.empty?
             end
 
             @tags
@@ -80,6 +85,7 @@ module Datadog
 
             @tags[Git::TAG_TAG] = Utils::Git.normalize_ref(@tags[Git::TAG_TAG])
             @tags[Git::TAG_BRANCH] = Utils::Git.normalize_ref(@tags[Git::TAG_BRANCH])
+            @tags[Git::TAG_PULL_REQUEST_BASE_BRANCH] = Utils::Git.normalize_ref(@tags[Git::TAG_PULL_REQUEST_BASE_BRANCH])
             @tags[Git::TAG_REPOSITORY_URL] = Datadog::Core::Utils::Url.filter_basic_auth(
               @tags[Git::TAG_REPOSITORY_URL]
             )

data/lib/datadog/ci/ext/environment/providers/appveyor.rb

@@ -74,6 +74,14 @@ module Datadog
               env["APPVEYOR_REPO_COMMIT_AUTHOR_EMAIL"]
             end
 
+            def git_commit_head_sha
+              env["APPVEYOR_PULL_REQUEST_HEAD_COMMIT"]
+            end
+
+            def pr_number
+              env["APPVEYOR_PULL_REQUEST_NUMBER"]
+            end
+
             def git_commit_message
               commit_message = env["APPVEYOR_REPO_COMMIT_MESSAGE"]
               if commit_message
@@ -85,7 +93,7 @@ module Datadog
 
             def git_pull_request_base_branch
               # from docs: build branch. For Pull Request commits it is base branch PR is merging into
-              env["APPVEYOR_REPO_BRANCH"]
+              env["APPVEYOR_REPO_BRANCH"] if env["APPVEYOR_PULL_REQUEST_HEAD_REPO_BRANCH"]
             end
 
             private

data/lib/datadog/ci/ext/environment/providers/azure.rb

@@ -80,6 +80,14 @@ module Datadog
               env["BUILD_SOURCEVERSIONMESSAGE"]
             end
 
+            def git_pull_request_base_branch
+              env["SYSTEM_PULLREQUEST_TARGETBRANCH"]
+            end
+
+            def pr_number
+              env["SYSTEM_PULLREQUEST_PULLREQUESTNUMBER"]
+            end
+
             def ci_env_vars
               {
                 "SYSTEM_TEAMPROJECTID" => env["SYSTEM_TEAMPROJECTID"],

data/lib/datadog/ci/ext/environment/providers/base.rb

@@ -105,6 +105,9 @@ module Datadog
             def git_commit_head_sha
             end
 
+            def pr_number
+            end
+
             private
 
             def set_branch_and_tag

data/lib/datadog/ci/ext/environment/providers/bitbucket.rb

@@ -65,6 +65,10 @@ module Datadog
               env["BITBUCKET_PR_DESTINATION_BRANCH"]
             end
 
+            def pr_number
+              env["BITBUCKET_PR_ID"]
+            end
+
             private
 
             def url

data/lib/datadog/ci/ext/environment/providers/bitrise.rb

@@ -79,6 +79,10 @@ module Datadog
               # For example, a pull request wants to merge the content of a branch into the branch main. In this case, this Env Var’s value is main.
               env["BITRISEIO_GIT_BRANCH_DEST"]
             end
+
+            def pr_number
+              env["BITRISE_PULL_REQUEST"]
+            end
           end
         end
       end

data/lib/datadog/ci/ext/environment/providers/buddy.rb

@@ -70,6 +70,10 @@ module Datadog
               # from docs: The name of the Git BASE branch of the currently run Pull Request
               env["BUDDY_RUN_PR_BASE_BRANCH"]
             end
+
+            def pr_number
+              env["BUDDY_RUN_PR_NO"]
+            end
           end
         end
       end

data/lib/datadog/ci/ext/environment/providers/buildkite.rb

@@ -85,6 +85,16 @@ module Datadog
               env["BUILDKITE_MESSAGE"]
             end
 
+            def git_pull_request_base_branch
+              env["BUILDKITE_PULL_REQUEST_BASE_BRANCH"]
+            end
+
+            def pr_number
+              return nil if env["BUILDKITE_PULL_REQUEST"] == "false"
+
+              env["BUILDKITE_PULL_REQUEST"]
+            end
+
             def ci_env_vars
               {
                 "BUILDKITE_BUILD_ID" => env["BUILDKITE_BUILD_ID"],

data/lib/datadog/ci/ext/environment/providers/circleci.rb

@@ -72,6 +72,10 @@ module Datadog
               env["BUILD_SOURCEVERSIONMESSAGE"]
             end
 
+            def pr_number
+              env["CIRCLE_PR_NUMBER"]
+            end
+
             def ci_env_vars
               {
                 "CIRCLE_WORKFLOW_ID" => env["CIRCLE_WORKFLOW_ID"],

data/lib/datadog/ci/ext/environment/providers/codefresh.rb

@@ -40,6 +40,14 @@ module Datadog
               env["CF_BRANCH"]
             end
 
+            def git_pull_request_base_branch
+              env["CF_PULL_REQUEST_TARGET"]
+            end
+
+            def pr_number
+              env["CF_PULL_REQUEST_NUMBER"]
+            end
+
             def ci_env_vars
               {
                 "CF_BUILD_ID" => env["CF_BUILD_ID"]

data/lib/datadog/ci/ext/environment/providers/drone.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+require "json"
+
+require_relative "base"
+
+module Datadog
+  module CI
+    module Ext
+      module Environment
+        module Providers
+          # Drone CI: https://drone.io/
+          # Environment variables docs: https://docs.drone.io/pipeline/environment/reference/
+          class Drone < Base
+            def self.handles?(env)
+              env.key?("DRONE")
+            end
+
+            def provider_name
+              Provider::DRONE
+            end
+
+            def job_name
+              env["DRONE_STEP_NAME"]
+            end
+
+            def pipeline_number
+              env["DRONE_BUILD_NUMBER"]
+            end
+
+            def pipeline_url
+              env["DRONE_BUILD_LINK"]
+            end
+
+            def stage_name
+              env["DRONE_STAGE_NAME"]
+            end
+
+            def workspace_path
+              env["DRONE_WORKSPACE"]
+            end
+
+            def git_repository_url
+              env["DRONE_GIT_HTTP_URL"]
+            end
+
+            def git_commit_sha
+              env["DRONE_COMMIT_SHA"]
+            end
+
+            def git_branch
+              env["DRONE_BRANCH"]
+            end
+
+            def git_tag
+              env["DRONE_TAG"]
+            end
+
+            def git_commit_author_name
+              env["DRONE_COMMIT_AUTHOR_NAME"]
+            end
+
+            def git_commit_author_email
+              env["DRONE_COMMIT_AUTHOR_EMAIL"]
+            end
+
+            def git_commit_message
+              env["DRONE_COMMIT_MESSAGE"]
+            end
+
+            def git_pull_request_base_branch
+              env["DRONE_TARGET_BRANCH"]
+            end
+
+            def pr_number
+              env["DRONE_PULL_REQUEST"]
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/datadog/ci/ext/environment/providers/github_actions.rb

@@ -77,8 +77,6 @@ module Datadog
             end
 
             def git_pull_request_base_branch
-              return nil if github_event_json.nil?
-
               env["GITHUB_BASE_REF"]
             end