fix(#49): github api rate limit in actions

Brewforge · Jan 12, 2024 · 5f5225c · 5f5225c
1 parent b610164
commit 5f5225c
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 10 deletions.
diff --git a/.github/actions/bump-casks/action.yml b/.github/actions/bump-casks/action.yml
@@ -14,7 +14,12 @@ runs:
       env:
         HOMEBREW_GITHUB_API_TOKEN: ${{ inputs.HOMEBREW_GITHUB_API_TOKEN }}
       shell: bash
+      # https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28#primary-rate-limit-for-authenticated-users
+      # Github Actions: 5000 requests per hour
+      # 1 * 60 * 24 = 1440 (because bump-pr will checkout a new branch and push to remote)
+      # 1 concurrent parallel jobs, 1 second delay between jobs.
       run: |
-        parallel -j0 --group --halt now,fail=1 \
-          './.github/actions/bump-casks/script.sh {}' \
-          ::: $(ls $(brew --repository brewforge/chinese)/Casks/*.rb | xargs -I {} basename {} .rb)
+        cd $(brew --repository brewforge/chinese)
+        parallel -j1 --delay 1s --group --halt now,fail=1 \
+          "$GITHUB_WORKSPACE/.github/actions/bump-casks/script.sh {}" \
+          ::: $(ls ./Casks/*.rb | xargs -I {} basename {} .rb)
diff --git a/.github/actions/bump-casks/script.sh b/.github/actions/bump-casks/script.sh
@@ -27,6 +27,6 @@ toV=${cleanCheck#*==>}
 
 echo "Updating $cask from $fromV to $toV"
 if [[ "$fromV" != "$toV" ]]; then
-  brew bump-cask-pr "$cask" --version "$toV" --verbose --force
+  brew bump-cask-pr "$cask" --version "$toV" --verbose --force --no-browse
 fi
 echo "Done for $cask"
diff --git a/.github/actions/bump-formulas/action.yml b/.github/actions/bump-formulas/action.yml
@@ -14,7 +14,12 @@ runs:
       env:
         HOMEBREW_GITHUB_API_TOKEN: ${{ inputs.HOMEBREW_GITHUB_API_TOKEN }}
       shell: bash
+      # https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28#primary-rate-limit-for-authenticated-users
+      # Github Actions: 5000 requests per hour
+      # 1 * 60 * 24 = 1440 (because bump-pr will checkout a new branch and push to remote)
+      # 1 concurrent parallel jobs, 1 second delay between jobs.
       run: |
-        parallel -j0 --group --halt now,fail=1 \
-          './.github/actions/bump-formulas/script.sh {}' \
-          ::: $(ls $(brew --repository brewforge/chinese)/Formula/*.rb | xargs -I {} basename {} .rb)
+        cd $(brew --repository brewforge/chinese)
+        parallel -j1 --delay 1s --group --halt now,fail=1 \
+          "$GITHUB_WORKSPACE/.github/actions/bump-formulas/script.sh {}" \
+          ::: $(ls ./Formula/*.rb | xargs -I {} basename {} .rb)
diff --git a/.github/actions/bump-formulas/script.sh b/.github/actions/bump-formulas/script.sh
@@ -27,6 +27,6 @@ toV=${cleanCheck#*==>}
 
 echo "Updating $formula from $fromV to $toV"
 if [[ "$fromV" != "$toV" ]]; then
-  brew bump-formula-pr "$formula" --version "$toV" --verbose --force
+  brew bump-formula-pr "$formula" --version "$toV" --verbose --force --no-browse
 fi
 echo "Done for $formula"
diff --git a/.github/actions/homebrew-env/action.yml b/.github/actions/homebrew-env/action.yml
@@ -17,13 +17,16 @@ runs:
         cask: true
         test-bot: false
 
-    - name: Add parallel cli
+    - name: Necessary clis
       shell: bash
       run: |
         brew install parallel
         # makesure brew livecheck works
         brew livecheck parallel
 
+        brew install coreutils
+        brew link --overwrite coreutils
+
     - name: Add Tap
       env:
         HOMEBREW_GITHUB_API_TOKEN: ${{ inputs.HOMEBREW_GITHUB_API_TOKEN }}
@@ -32,3 +35,6 @@ runs:
         brew tap brewforge/chinese
         brew update
         brew install ripgrep sd
+        brew install-bundler-gems --groups "audit,style"
+
+        brew update-reset $(brew --repository brewforge/chinese)
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -2,7 +2,7 @@ name: CI
 
 on:
   pull_request:
-    branches: [main, bump-*]
+    branches: [main]
   push:
     branches: [fix-*]
   schedule: