admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Merge pull request #14670 from MikeMcQuaid/analytics_on_request_bool

Browse Source 
utils/analytics: make on_request a proper boolean.
This commit is contained in:
Mike McQuaid 2023-02-17 03:41:18 +00:00 committed by GitHub
commit b55078690c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Library/Homebrew/utils/analytics.rb
View File

@ -76,8 +76,11 @@ module Utils
additional_tags: T::Hash[Symbol, T.untyped]).void
}
def report_influx(measurement, package_and_options, on_request, additional_tags = {})
# convert on_request to a boolean
on_request = on_request ? true : false
# Append general information to device information
tags = additional_tags.merge(package_and_options: package_and_options, on_request: !on_request.nil?)
tags = additional_tags.merge(package_and_options: package_and_options, on_request: on_request)
.compact
.map { |k, v| "#{k}=#{v.to_s.sub(" ", "\\ ")}" } # convert to key/value parameters
.join(",")