move fmt to github hosted

.github/actions/fmt/Dockerfile

@@ -0,0 +1,5 @@
+FROM ghcr.io/oneflow-inc/oneflow-fmt:latest
+RUN python3 -m pip install black==19.10b0
+COPY entrypoint.sh /entrypoint.sh
+RUN chmod +x /entrypoint.sh
+ENTRYPOINT ["/entrypoint.sh"]

.github/actions/fmt/action.yml

@@ -3,17 +3,5 @@ inputs:
     description: 'Placeholder'
     default: 'Placeholder'
 runs:
-  using: "composite"
-  steps:
-    - name: Check license (please run 'make of_format' if failed)
-      shell: bash
-      run: |
-        python3 ci/check/run_license_format.py -i oneflow -c
-    - name: Check C++ format (please run 'make of_format' if failed)
-      shell: bash
-      run: |
-        python3 ci/check/run_clang_format.py --clang_format_binary clang-format --source_dir oneflow
-    - name: Check python format (please run 'make of_format' if failed)
-      shell: bash
-      run: |
-        docker run --rm -v $PWD:$PWD --entrypoint "/bin/sh" cytopia/black@sha256:ea649b81bf6a64aaa34f4abba803da83528135b5d12bed9b485bcabb37480d33 -c "python3 $PWD/ci/check/run_py_format.py --source_dir $PWD"
+  using: 'docker'
+  image: 'Dockerfile'

.github/actions/fmt/entrypoint.sh

@@ -0,0 +1,8 @@
+#!/bin/sh -l
+
+set -ex
+python3 ci/check/run_license_format.py -i oneflow -c
+python3 ci/check/run_clang_format.py --clang_format_binary clang-format --source_dir oneflow
+python3 ci/check/run_py_format.py --source_dir $PWD
+time=$(date)
+echo "::set-output name=time::$time"

.github/workflows/test.yml

@@ -15,6 +15,15 @@ on:
         required: false
 jobs:
 
+  check_license_and_format:
+
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.draft == false
+    steps:
+    - uses: actions/checkout@v2
+    - name: Check license and format (please run 'make of_format' if failed)
+      uses: ./.github/actions/fmt
+
   build_and_test:
 
     runs-on: [self-hosted, linux, gpu]
@@ -24,8 +33,6 @@ jobs:
       run: |
         docker run --rm -v $PWD:/p -w /p busybox chmod -R o+w .
     - uses: actions/checkout@v2
-    - name: Check license and format (please run 'make of_format' if failed)
-      uses: ./.github/actions/fmt
     - name: Setup submodule
       uses: ./.github/actions/setup
     - name: Build OneFlow

ci/build/make.sh

@@ -25,13 +25,12 @@ docker build -f $src_dir/docker/package/manylinux/Dockerfile \
 
 cd -
 
-docker run --rm \
-    -v $tmp_dir:/ci-tmp \
-    -w $tmp_dir:/ci-tmp busybox rm -rf /ci-tmp/wheelhouse
-
 # build function
 function build() {
     set -x
+    docker run --rm \
+        -v $tmp_dir:/ci-tmp \
+        -w $tmp_dir:/ci-tmp busybox rm -rf /ci-tmp/wheelhouse
     docker run \
         $docker_proxy_run_args \
         --rm $docker_it \
@@ -41,6 +40,7 @@ function build() {
         "$docker_tag" \
         /oneflow-src/docker/package/manylinux/build_wheel.sh \
             --python3.6 \
+            --house-dir /ci-tmp/wheelhouse \
             --package-name oneflow${package_appendix} \
             $extra_oneflow_cmake_args
 }

docker/ci/fmt/Dockerfile

@@ -0,0 +1,3 @@
+FROM python:3.7
+RUN curl https://oneflow-static.oss-cn-beijing.aliyuncs.com/bin/clang-format -o /usr/local/bin/clang-format && chmod +x /usr/local/bin/clang-format
+RUN apt update && apt install -y libncurses5

docker/ci/fmt/build.sh

@@ -0,0 +1,3 @@
+set -ex
+cd docker/ci/fmt
+docker build -t oneflow-fmt .

docker/package/manylinux/build_wheel.sh

@@ -78,7 +78,10 @@ ONEFLOW_BUILD_DIR=$CACHE_DIR/build-oneflow
 
 function cleanup {
   set -x
-  rm  -rf tmp_wheel
+  rm -rf $ONEFLOW_BUILD_DIR/python_scripts/oneflow/*.so
+  rm -rf build/bdist.linux-x86_64
+  rm -rf build/lib
+  rm -rf tmp_wheel
 }
 
 for PY_VER in ${PY_VERS[@]}
@@ -92,9 +95,7 @@ do
     fi
     PY_ROOT=/opt/python/${PY_ABI}
     PY_BIN=${PY_ROOT}/bin/python
-    rm -rf $ONEFLOW_BUILD_DIR/python_scripts/oneflow/*.so
-    rm -rf $ONEFLOW_SRC_DIR/build/bdist.linux-x86_64
-    rm -rf $ONEFLOW_SRC_DIR/build/lib
+    cleanup
     cmake -DTHIRD_PARTY=OFF -DONEFLOW=ON \
         -DCMAKE_EXPORT_COMPILE_COMMANDS=1 \
         $COMMON_CMAKE_ARGS \