Merge pull request #14821 from issyl0/support-date-ranges-in-pr-revie…

…ws-query

dev-cmd/contributions: Support date ranges in PR reviews query

Library/Homebrew/dev-cmd/contributions.rb

@@ -175,7 +175,7 @@ def scan_repositories(repos, person, args)
         commits:       GitHub.repo_commit_count_for_user(repo_full_name, person, args),
         coauthorships: git_log_trailers_cmd(T.must(repo_path), person, "Co-authored-by", args),
         signoffs:      git_log_trailers_cmd(T.must(repo_path), person, "Signed-off-by", args),
-        reviews:       GitHub.count_issues("", is: "pr", repo: repo_full_name, reviewed_by: person),
+        reviews:       GitHub.count_issues("", is: "pr", repo: repo_full_name, reviewed_by: person, args: args),
       }
     end
 

Library/Homebrew/utils/github.rb

@@ -159,11 +159,21 @@ def private_repo?(full_name)
   def search_query_string(*main_params, **qualifiers)
     params = main_params
 
-    params += qualifiers.flat_map do |key, value|
+    if (args = qualifiers.fetch(:args, nil))
+      params << if args.from && args.to
+        "created:#{args.from}..#{args.to}"
+      elsif args.from
+        "created:>=#{args.from}"
+      elsif args.to
+        "created:<=#{args.to}"
+      end
+    end
+
+    params += qualifiers.except(:args).flat_map do |key, value|
       Array(value).map { |v| "#{key.to_s.tr("_", "-")}:#{v}" }
     end
 
-    "q=#{URI.encode_www_form_component(params.join(" "))}&per_page=100"
+    "q=#{URI.encode_www_form_component(params.compact.join(" "))}&per_page=100"
   end
 
   def url_to(*subroutes)