Merge pull request #14647 from MikeMcQuaid/report_influxdb_default

analytics: report to InfluxDB by default.
Homebrew · Feb 15, 2023 · e42cae1 · e42cae1
2 parents dbd9f72 + 769a8c5
commit e42cae1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/Library/Homebrew/test/utils/analytics_spec.rb b/Library/Homebrew/test/utils/analytics_spec.rb
@@ -162,7 +162,6 @@
       ENV.delete("HOMEBREW_NO_ANALYTICS_THIS_RUN")
       ENV.delete("HOMEBREW_NO_ANALYTICS")
       ENV["HOMEBREW_ANALYTICS_DEBUG"] = "true"
-      ENV["HOMEBREW_ANALYTICS_ENABLE_INFLUX"] = "true"
       ENV["HOMEBREW_DEVELOPER"] = "1"
       expect(described_class).to receive(:deferred_curl).once
       described_class.report_influx(:install, action, true, developer: true, CI: true)

diff --git a/Library/Homebrew/utils/analytics.rb b/Library/Homebrew/utils/analytics.rb
@@ -76,8 +76,6 @@ def report_google(type, metadata = {})
                additional_tags: T::Hash[Symbol, T.untyped]).void
       }
       def report_influx(measurement, package_and_options, on_request, additional_tags = {})
-        return unless ENV["HOMEBREW_ANALYTICS_ENABLE_INFLUX"]
-
         # Append general information to device information
         tags = additional_tags.merge(package_and_options: package_and_options, on_request: !on_request.nil?)
                               .compact_blank