Fix code formatter bugs (#1247)

* Change permission to read in slash-command-dispatch.yml * Change permission to none in slash-command-dispatch.yml * Change trigger event from push to PR in autopep8.yml
aimclub · Jan 10, 2024 · dcc7ff5 · dcc7ff5
1 parent 8fb75e2
commit dcc7ff5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/.github/workflows/autopep8.yml b/.github/workflows/autopep8.yml
@@ -1,5 +1,5 @@
 name: Format python code with autopep8
-on: push
+on: pull_request
 jobs:
   autopep8:
     runs-on: ubuntu-latest
@@ -10,29 +10,24 @@ jobs:
         uses: peter-evans/autopep8@v2
         with:
           args: --exit-code --max-line-length 120 --recursive --in-place --aggressive --aggressive .
-      - name: Get Pull Request Number
-        id: pr
-        run: echo "::set-output name=pull_request_number::$(gh pr view --json number -q .number || echo "")"
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       - name: Find Comment
         uses: peter-evans/find-comment@v2
         id: fc
         with:
-          issue-number: ${{ steps.pr.outputs.pull_request_number }}
+          issue-number: ${{ github.event.pull_request.number }}
           comment-author: "github-actions[bot]"
       - name: Create comment if autopep8 made NO changes
         if: ${{ steps.fc.outputs.comment-id == '' && steps.autopep8.outputs.exit-code != 2}}
         uses: peter-evans/create-or-update-comment@v3
         with:
-          issue-number: ${{ steps.pr.outputs.pull_request_number }}
+          issue-number: ${{ github.event.pull_request.number }}
           body: |
             Code has no PEP8 errors!
       - name: Create comment if autopep8 made changes
         if: ${{ steps.fc.outputs.comment-id == '' && steps.autopep8.outputs.exit-code == 2}}
         uses: peter-evans/create-or-update-comment@v3
         with:
-          issue-number: ${{ steps.pr.outputs.pull_request_number }}
+          issue-number: ${{ github.event.pull_request.number }}
           body: |
             Code in this pull request contains PEP8 errors, please write the `/fix-pep8` command in the comments below to create commit with automatic fixes.
       - name: Update comment if NOT fixed

diff --git a/.github/workflows/slash-command-dispatch.yml b/.github/workflows/slash-command-dispatch.yml
@@ -12,5 +12,5 @@ jobs:
           token: ${{ secrets.REPO_ACCESS_TOKEN }}
           reactions: true
           commands: fix-pep8
-          permission: read
+          permission: none
           issue-type: pull-request