refine github rl workflow (#1867)

* rl workflow

* fix

* lint

* update

* revert

Author: hhaAndroid

.github/workflows/unit_test.yaml

@@ -3,7 +3,6 @@ on:
   pull_request:
     branches:
       - "main"
-      - "refactor"
     paths-ignore:
       - "docs/**"
       - "**.md"
@@ -21,8 +20,44 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
+  detect_changes:
+    runs-on: ubuntu-latest
+    outputs:
+      only_rl: ${{ steps.filter.outputs.only_rl }}
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+      - id: filter
+        name: Detect changed paths
+        run: |
+          if ! changed_files="$(git diff --name-only "${{ github.event.pull_request.base.sha }}" "${{ github.event.pull_request.head.sha }}" 2>/dev/null)"; then
+            changed_files="$(git diff --name-only HEAD^1 HEAD)"
+          fi
+          only_rl=true
+
+          if [ -z "$changed_files" ]; then
+            only_rl=false
+          else
+            while IFS= read -r file; do
+              case "$file" in
+                xtuner/v1/rl/*)
+                  ;;
+                *)
+                  only_rl=false
+                  break
+                  ;;
+              esac
+            done <<< "$changed_files"
+          fi
+
+          echo "$changed_files"
+          echo "only_rl=$only_rl"
+          echo "only_rl=$only_rl" >> "$GITHUB_OUTPUT"
+
   lint:
     runs-on: ubuntu-latest
+    needs: detect_changes
     steps:
       - uses: actions/checkout@v2
       - name: Set up Python 3.12
@@ -34,11 +69,19 @@ jobs:
           pip install pre-commit
           pre-commit install
       - name: Linting
-        run: pre-commit run --files $(find xtuner/v1)
+        run: |
+          if [ "${{ needs.detect_changes.outputs.only_rl }}" = "true" ]; then
+            lint_target="xtuner/v1/rl"
+          else
+            lint_target="xtuner/v1"
+          fi
+          pre-commit run --files $(find "$lint_target" -type f)
 
   unit_test:
     runs-on: [h_cluster]
-    needs: lint
+    needs:
+      - detect_changes
+      - lint
     steps:
     - name: mask env
       run: |
@@ -50,4 +93,10 @@ jobs:
     - name: unit-test
       run: |
         export PYTHONPYCACHEPREFIX=/tmp
-        python ci/scripts/xtuner_unittest.py "$IMAGE_INNER/$IMAGE" "source ${{env.WORKSPACE_PREFIX}}/BASE_ENV.sh;source ci/scripts/CI_ENV.sh" "pytest tests"
+        if [ "${{ needs.detect_changes.outputs.only_rl }}" = "true" ]; then
+          test_target="tests/rl"
+        else
+          test_target="tests"
+        fi
+        echo "test_target=$test_target"
+        python ci/scripts/xtuner_unittest.py "$IMAGE_INNER/$IMAGE" "source ${{env.WORKSPACE_PREFIX}}/BASE_ENV.sh;source ci/scripts/CI_ENV.sh" "pytest $test_target"