HyukjinKwon commented on a change in pull request #34163:
URL: https://github.com/apache/spark/pull/34163#discussion_r726671839



##########
File path: .github/workflows/stale.yml
##########
@@ -21,23 +21,22 @@ name: Close stale PRs
 
 on:
   schedule:
-  - cron: "0 0 * * *"
+    - cron: "0 0 * * *"
 
 jobs:
   stale:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/[email protected]
-      with:
-        repo-token: ${{ secrets.GITHUB_TOKEN }}
-        stale-pr-message: >
-          We're closing this PR because it hasn't been updated in a while.
-          This isn't a judgement on the merit of the PR in any way. It's just
-          a way of keeping the PR queue manageable.
+      - uses: actions/stale@c201d45ef4b0ccbd3bb0616f93bae13e73d0a080 # 
[email protected]
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          stale-pr-message: >
+            We're closing this PR because it hasn't been updated in a while.
+            This isn't a judgement on the merit of the PR in any way. It's just
+            a way of keeping the PR queue manageable.
 
-          If you'd like to revive this PR, please reopen it and ask a
-          committer to remove the Stale tag!
-        days-before-stale: 100
-        # Setting this to 0 is the same as setting it to 1.
-        # See: https://github.com/actions/stale/issues/28
-        days-before-close: 0
+            If you'd like to revive this PR, please reopen it and ask a 
committer to remove the Stale tag!
+          days-before-stale: 100
+          # Setting this to 0 is the same as setting it to 1.
+          # See: https://github.com/actions/stale/issues/28
+          days-before-close: 0

Review comment:
       ```suggestion
             If you'd like to revive this PR, please reopen it and ask a 
committer to remove the Stale tag!
           days-before-stale: 100
           # Setting this to 0 is the same as setting it to 1.
           # See: https://github.com/actions/stale/issues/28
           days-before-close: 0
   ```

##########
File path: .github/workflows/update_build_status.yml
##########
@@ -21,15 +21,15 @@ name: Update build status workflow
 
 on:
   schedule:
-  - cron: "*/15 * * * *"
+    - cron: "*/15 * * * *"

Review comment:
       ```suggestion
     - cron: "*/15 * * * *"
   ```

##########
File path: .github/workflows/test_report.yml
##########
@@ -20,25 +20,25 @@
 name: Report test results
 on:
   workflow_run:
-    workflows: ["Build and test"]
+    workflows: [ "Build and test" ]

Review comment:
       ```suggestion
       workflows: ["Build and test"]
   ```

##########
File path: .github/workflows/notify_test_workflow.yml
##########
@@ -25,15 +25,15 @@
 name: On pull request update
 on:
   pull_request_target:
-    types: [opened, reopened, synchronize]
+    types: [ opened, reopened, synchronize ]

Review comment:
       ```suggestion
       types: [opened, reopened, synchronize]
   ```

##########
File path: .github/workflows/test_report.yml
##########
@@ -20,25 +20,25 @@
 name: Report test results
 on:
   workflow_run:
-    workflows: ["Build and test"]
+    workflows: [ "Build and test" ]
     types:
       - completed
 
 jobs:
   test_report:
     runs-on: ubuntu-latest
     steps:
-    - name: Download test results to report
-      uses: dawidd6/action-download-artifact@v2
-      with:
-        github_token: ${{ secrets.GITHUB_TOKEN }}
-        workflow: ${{ github.event.workflow_run.workflow_id }}
-        commit: ${{ github.event.workflow_run.head_commit.id }}
-        workflow_conclusion: completed
-    - name: Publish test report
-      uses: scacap/action-surefire-report@v1
-      with:
-        check_name: Report test results
-        github_token: ${{ secrets.GITHUB_TOKEN }}
-        report_paths: "**/target/test-reports/*.xml"
-        commit: ${{ github.event.workflow_run.head_commit.id }}
+      - name: Download test results to report
+        uses: 
dawidd6/action-download-artifact@6f8f427fb41886a66b82ea11a5a15d1454c79415 # 
pin@v2
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          workflow: ${{ github.event.workflow_run.workflow_id }}
+          commit: ${{ github.event.workflow_run.head_commit.id }}
+          workflow_conclusion: completed
+      - name: Publish test report
+        uses: 
scacap/action-surefire-report@482f012643ed0560e23ef605a79e8e87ca081648 # pin@v1
+        with:
+          check_name: Report test results
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          report_paths: "**/target/test-reports/*.xml"
+          commit: ${{ github.event.workflow_run.head_commit.id }}

Review comment:
       ```suggestion
       - name: Download test results to report
         uses: 
dawidd6/action-download-artifact@6f8f427fb41886a66b82ea11a5a15d1454c79415 # 
pin@v2
         with:
           github_token: ${{ secrets.GITHUB_TOKEN }}
           workflow: ${{ github.event.workflow_run.workflow_id }}
           commit: ${{ github.event.workflow_run.head_commit.id }}
           workflow_conclusion: completed
       - name: Publish test report
         uses: 
scacap/action-surefire-report@482f012643ed0560e23ef605a79e8e87ca081648 # pin@v1
         with:
           check_name: Report test results
           github_token: ${{ secrets.GITHUB_TOKEN }}
           report_paths: "**/target/test-reports/*.xml"
           commit: ${{ github.event.workflow_run.head_commit.id }}
   ```

##########
File path: .github/workflows/stale.yml
##########
@@ -21,23 +21,22 @@ name: Close stale PRs
 
 on:
   schedule:
-  - cron: "0 0 * * *"
+    - cron: "0 0 * * *"

Review comment:
       ```suggestion
     - cron: "0 0 * * *"
   ```
   

##########
File path: .github/workflows/stale.yml
##########
@@ -21,23 +21,22 @@ name: Close stale PRs
 
 on:
   schedule:
-  - cron: "0 0 * * *"
+    - cron: "0 0 * * *"
 
 jobs:
   stale:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/[email protected]
-      with:
-        repo-token: ${{ secrets.GITHUB_TOKEN }}
-        stale-pr-message: >
-          We're closing this PR because it hasn't been updated in a while.
-          This isn't a judgement on the merit of the PR in any way. It's just
-          a way of keeping the PR queue manageable.
+      - uses: actions/stale@c201d45ef4b0ccbd3bb0616f93bae13e73d0a080 # 
[email protected]
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          stale-pr-message: >
+            We're closing this PR because it hasn't been updated in a while.
+            This isn't a judgement on the merit of the PR in any way. It's just
+            a way of keeping the PR queue manageable.

Review comment:
       ```suggestion
       - uses: actions/stale@c201d45ef4b0ccbd3bb0616f93bae13e73d0a080 # 
[email protected]
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
           stale-pr-message: >
             We're closing this PR because it hasn't been updated in a while.
             This isn't a judgement on the merit of the PR in any way. It's just
             a way of keeping the PR queue manageable.
   ```




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to