Update PR when NOTICES changes. (#9735)

* Update PR when NOTICES changes.

Author: granluo

.github/workflows/notice_generation.yml

@@ -39,5 +39,11 @@ jobs:
     - name: Create a pull request
       run: |
         gem install octokit
-        ruby scripts/create_pull_request.rb --repo-root ${GITHUB_WORKSPACE} --repo-token ${{ secrets.GITHUB_TOKEN }} --notices-path  ${{ env.NOTICES_PATH }}
+        ruby scripts/create_pull_request.rb \
+        --repo-root ${GITHUB_WORKSPACE} \
+        --repo-token ${{ secrets.GITHUB_TOKEN }} \
+        --target-path  ${{ env.NOTICES_PATH }} \
+        --pr-title "NOTICES Change" \
+        --pr-body "NOTICES Change is detected." \
+        --commit-comment "NOTICES change."
       shell: bash

scripts/create_pull_request.rb

@@ -18,37 +18,53 @@
 require 'optparse'
 
 @options = {
-  repo_root: "./"
+  repo_root: "./",
+  pr_title: "Autogenerated PR",
+  pr_body: "",
+  base_branch: "generated-branch-for-pr-"+Time.new.strftime("%Y%m%d%H%M%S"),
+  commit_comment: "Auto generated commit."
 }
 begin
   OptionParser.new do |opts|
     opts.banner = "Usage: create_pull_request.rb [options]"
     opts.on('-r', '--repo-root REPO_ROOT', 'Root path of the repo dir.') { |v| @options[:repo_root] = v }
     opts.on('-t', '--repo-token REPO_TOKEN', 'Token with write access') { |v| @options[:repo_token] = v }
-    opts.on('-n', '--notices-path NOTICES_PATH', 'Path of NOTICES file') { |v| @options[:notices_path] = v }
+    opts.on('-n', '--target-path PATH', 'Path of targeted file or dir') { |v| @options[:target_path] = v }
+    opts.on('--pr-title PR_TITLE', 'Title of a PR') { |v| @options[:pr_title] = v }
+    opts.on('--pr-body PR_BODY', 'Body of a PR') { |v| @options[:pr_body] = v }
+    opts.on('--base-branch', 'A new branch will be generated if not specified or the branch does not exist.') { |v| @options[:base_branch] = v }
+    opts.on('--commit-comment', 'Commit comment') { |v| @options[:commit_comment] = v }
   end.parse!
 
-  raise OptionParser::MissingArgument if @options[:repo_token].nil? || @options[:notices_path].nil?
+  raise OptionParser::MissingArgument if @options[:repo_token].nil? || @options[:target_path].nil?
 rescue OptionParser::MissingArgument
-  puts "Notices path, `--notices-path`, should be specified. " if @options[:notices_path].nil?
+  puts "target path, `--target-path`, should be specified. " if @options[:target_path].nil?
   puts "A token ,`--repo-token`, should be provided for creating a pull request." if @options[:repo_token].nil?
   raise
 end
 
 REPO_ROOT=@options[:repo_root]
 ACCESS_TOKEN=@options[:repo_token]
-NOTICES_PATH=@options[:notices_path]
+TARGET_PATH=@options[:target_path]
+PR_TITLE=@options[:pr_title]
+PR_BODY=@options[:pr_boday]
+BASE_BRANCH=@options[:base_branch]
+COMMIT_COMMENT=@options[:commit_comment]
 
-def generate_pr_for_notices_changes(repo_root:, notices_path:)
-  system("cd #{repo_root}")
-  system("git checkout -b notices_diff_detected\n git add CoreOnly/NOTICES\n git commit -m \"NOTICES diff detected.\"\n git push -u origin notices_diff_detected")
+def generate_pr_for_target_changes(repo_root:, target_path:)
+  system("cd #{REPO_ROOT}\ngit checkout -b #{BASE_BRANCH}\n")
+  if `git diff #{TARGET_PATH}`==""
+    puts "The file, #{TARGET_PATH}, has no changes."
+    return
+  end
+  system("git add #{TARGET_PATH}\ngit commit -m \"#{COMMIT_COMMENT}\"\n git push -u origin #{BASE_BRANCH}")
   client = Octokit::Client.new(access_token: ACCESS_TOKEN)
-  client.create_pull_request("firebase/firebase-ios-sdk", "master", "notices_diff_detected", "Pull Request title", "Pull Request body")
+  client.create_pull_request("firebase/firebase-ios-sdk", "master", BASE_BRANCH, PR_TITLE, PR_BODY)
 end
 
 
 def main()
-  generate_pr_for_notices_changes(repo_root: REPO_ROOT, notices_path: NOTICES_PATH)
+  generate_pr_for_target_changes(repo_root: REPO_ROOT, target_path: TARGET_PATH)
 end
 
 main()