Update GitHub action and pre-commit settings (#196)

* Update GitHub action and pre-commit settings

* Update GitHub action and pre-commit settings (#198)

.github/reviewer_list.yml

+addReviewers: true
+
+addAssignees: author
+
+numberOfReviewers: 1
+
+reviewers:
+  - frankleeeee
+  - kurisusnowdeng

.github/workflows/build.yml → .github/workflows/PR_CI.yml

-name: Build
-on:
-  pull_request:
-    types: [review_requested, synchronize]
-    branches:
-      - "*"
+name: Pull Request CI and Build
+
+on: pull_request
 
 jobs:
+  assign_reviewer:
+    name: Assign Reviewer for PR
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.draft == false && github.base_ref == 'main' && github.event.pull_request.base.repo.full_name == 'hpcaitech/ColossalAI' && toJson(github.event.pull_request.requested_reviewers) == '[]'
+    steps:
+      - uses: kentaro-m/auto-assign-action@v1.2.1
+        with:
+          configuration-path: '.github/reviewer_list.yml'
+
   build:
-    name: Build and test Colossal-AI
+    name: Build and Test Colossal-AI
+    if: ${{ always() }} && github.event.pull_request.draft == false && github.base_ref == 'main' && github.event.pull_request.base.repo.full_name == 'hpcaitech/ColossalAI'
+    needs: [assign_reviewer]
     runs-on: [self-hosted, gpu]
     container:
       image: nvcr.io/nvidia/pytorch:21.07-py3
       options: --gpus all --rm --ipc=host -v /data/cifar-10:/data/cifar-10
-    timeout-minutes: 1200
-    if: github.event.pull_request.draft == false && github.base_ref == 'main' && github.event.pull_request.base.repo.full_name == 'hpcaitech/ColossalAI' && toJson(github.event.pull_request.requested_reviewers) != '[]'
+    timeout-minutes: 20
     steps:
       - name: Setup Environment
         run: |
@@ -26,6 +33,7 @@ jobs:
       - uses: actions/checkout@v2
       - name: Install Colossal-AI      
         run: |
+          pip install -r requirements/requirements.txt
           pip install -v --no-cache-dir --global-option="--cuda_ext" .
       - name: Unit Testing
         run: |
@@ -33,8 +41,43 @@ jobs:
         env:
           DATA: /data/cifar-10
 
+  format_check:
+    name: Format Check
+    if: ${{ always() }} && github.event.pull_request.draft == false && github.base_ref == 'main' && github.event.pull_request.base.repo.full_name == 'hpcaitech/ColossalAI'
+    needs: [assign_reviewer, build]
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v2
+        with:
+          ref: ${{ github.head_ref }}
 
+      - name: autoyapf
+        id: autoyapf
+        uses: mritunjaysharma394/autoyapf@v2
+        with:
+          args: --style google --recursive --in-place .
 
+      - name: Check for modified files
+        id: git-check
+        run: echo ::set-output name=modified::$(if git diff-index --quiet HEAD --; then echo "false"; else echo "true"; fi)
       
+      - name: Push changes
+        if: steps.git-check.outputs.modified == 'true'
+        run: |
+          git config --global user.name 'github-actions' 
+          git config --global user.email 'github-actions@github.com' 
+          git remote set-url origin https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/${{ github.repository }}
+          git commit -am "Automated autoyapf fixes"
       
-
+      - name: Create Pull Request
+        # if: steps.format.outputs.has-changes == 'true'
+        uses: peter-evans/create-pull-request@v3
+        with:
+          title: '[Bot] Automated PR to fix formatting errors'
+          body: |
+            Automated PR to fix formatting errors
+          committer: GitHub <noreply@github.com>
+          author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
+          assignees: ${{ github.actor }}
+          reviewers: frankleeeee

.github/workflows/close_inactive.yml

@@ -13,10 +13,12 @@ jobs:
       - uses: actions/stale@v3
         with:
           days-before-issue-stale: 14
-          days-before-issue-close: 14
+          days-before-issue-close: -1
           stale-issue-label: "stale"
           stale-issue-message: "This issue is stale because it has been open for 14 days with no activity."
-          close-issue-message: "This issue was closed because it has been inactive for 14 days since being marked as stale."
+#           close-issue-message: "This issue was closed because it has been inactive for 14 days since being marked as stale."
           days-before-pr-stale: 14
-          days-before-pr-close: 14
+          days-before-pr-close: -1
+          stale-pr-message: "This PR is stale because it has been open for 14 days with no activity."
+#           close-pr-message: "This PR was closed because it has been inactive for 14 days since being marked as stale."
           repo-token: ${{ secrets.GITHUB_TOKEN }}

.pre-commit-config.yaml

+repos:
+  - repo: https://github.com/pre-commit/mirrors-yapf
+    rev: v0.32.0
+    hooks:
+    - id: yapf
+      args: ['--style=google', '--parallel', '--in-place']
+  - repo: https://github.com/pycqa/flake8
+    rev: ''
+    hooks:
+    - id: flake8
+  - repo: https://github.com/pre-commit/mirrors-clang-format
+    rev: v13.0.0
+    hooks:
+    - id: clang-format

README.md

@@ -5,7 +5,7 @@
 <div align="center">
    <h3> <a href="https://arxiv.org/abs/2110.14883"> Paper </a> | <a href="https://www.colossalai.org/"> Documentation </a> | <a href="https://github.com/hpcaitech/ColossalAI/discussions"> Forum </a> | <a href="https://medium.com/@hpcaitech"> Blog </a></h3>
    
-   [![Build](https://github.com/hpcaitech/ColossalAI/actions/workflows/build.yml/badge.svg)](https://github.com/hpcaitech/ColossalAI/actions/workflows/build.yml)
+   [![Build](https://github.com/hpcaitech/ColossalAI/actions/workflows/PR_CI.yml/badge.svg)](https://github.com/hpcaitech/ColossalAI/actions/workflows/PR_CI.yml)
    [![Documentation](https://readthedocs.org/projects/colossalai/badge/?version=latest)](https://colossalai.readthedocs.io/en/latest/?badge=latest)
 </div>
 An integrated large-scale model training system with efficient parallelization techniques.

requirements/requirements.txt

@@ -5,3 +5,4 @@ tqdm
 psutil
 tensorboard
 packaging
+pre-commit