Merge pull request #5291 from cakebaker/ci_fix_checkscripts_yml

ci: remove committing from CheckScripts.yml

.github/workflows/CheckScripts.yml

@@ -41,14 +41,9 @@ jobs:
 
   shell_fmt:
     name: ShellScript/Format
-    # no need to run in pr events
-    # shfmt will be performed on main branch when the PR is merged
-    if: github.event_name != 'pull_request'
     runs-on: ubuntu-latest
-    needs: [ shell_check ]
     permissions:
-      contents: write
-      pull-requests: write
+      contents: read
     steps:
       - uses: actions/checkout@v4
       - name: Setup shfmt
@@ -56,19 +51,6 @@ jobs:
       - name: Run shfmt
         shell: bash
         run: |
-          # show differs first for every files that need to be formatted
           # fmt options: bash syntax, 4 spaces indent, indent for switch-case
           echo "## show the differences between formatted and original scripts..."
           find ${{ env.SCRIPT_DIR }} -name "*.sh" -print0 | xargs -0 shfmt -ln=bash -i 4 -ci -d || true
-          # perform a shell format
-          echo "## perform a shell format..."
-          # ignore the error code because `-d` will always return false when the file has difference
-          find ${{ env.SCRIPT_DIR }} -name "*.sh" -print0 | xargs -0 shfmt -ln=bash -i 4 -ci -w
-      - name: Commit any changes
-        uses: EndBug/add-and-commit@v9
-        with:
-          default_author: github_actions
-          message: "style: auto format by CI (shfmt)"
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-