From 42a2d38f47ec26953f414c9eda0b720c07b7fdcc Mon Sep 17 00:00:00 2001 From: WeiJie_Hong <1462519292@qq.com> Date: Mon, 2 Feb 2026 12:03:02 +0800 Subject: [PATCH] [CI/Build] Fixed bug related to conflicts in the code inspection tool (#169) Signed-off-by: WeiJie_Hong <1462519292@qq.com> --- .github/workflows/pylint-check.yml | 20 +++++++++---------- .pre-commit-config.yaml | 32 +++++++++++++++--------------- 2 files changed, 26 insertions(+), 26 deletions(-) diff --git a/.github/workflows/pylint-check.yml b/.github/workflows/pylint-check.yml index 634fdbb..f8a60e5 100644 --- a/.github/workflows/pylint-check.yml +++ b/.github/workflows/pylint-check.yml @@ -4,7 +4,6 @@ on: pull_request: push: branches: [ main ] - jobs: pylint-check: runs-on: ubuntu-latest @@ -27,9 +26,9 @@ jobs: shell: bash run: | if [ "${{ github.event_name }}" = "pull_request" ]; then - files=$(git diff --name-only origin/${{ github.base_ref }}...HEAD) + files=$(git diff --name-only --diff-filter=ACMRT origin/${{ github.base_ref }}...HEAD) else - files=$(git diff --name-only HEAD~1...HEAD) + files=$(git diff --name-only --diff-filter=ACMRT HEAD~1...HEAD) fi files=$(echo "$files" | grep '\.py$' || true) @@ -38,17 +37,18 @@ jobs: echo "$files" >> $GITHUB_OUTPUT echo "EOF" >> $GITHUB_OUTPUT + - name: Run ruff + if: steps.changed.outputs.files != '' + run: | + echo "${{ steps.changed.outputs.files }}" | xargs ruff check --fix + continue-on-error: true + - name: Run black if: steps.changed.outputs.files != '' run: | - black --check ${{ steps.changed.outputs.files }} + echo "${{ steps.changed.outputs.files }}" | xargs black --check - name: Run isort if: steps.changed.outputs.files != '' run: | - isort --check-only ${{ steps.changed.outputs.files }} - - - name: Run ruff - if: steps.changed.outputs.files != '' - run: | - ruff check ${{ steps.changed.outputs.files }} + echo "${{ steps.changed.outputs.files }}" | xargs isort --check-only --profile black \ No newline at end of file diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 3c11c09..3e12d21 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -21,6 +21,21 @@ repos: args: ["--maxkb=10240"] # Maximum allowed file size (10MB) # ----------------------- + # Python Linting (Ruff) + # Characteristics: Fast, suitable for CI, replaces flake8/isort/etc. + # ----------------------- + - repo: https://github.com/astral-sh/ruff-pre-commit + rev: v0.14.14 # Ruff version + hooks: + - id: ruff-check # Main linting checks + types_or: [ python, pyi ] # Check .py and .pyi files + args: [ --fix ] # Auto-fix fixable issues + # Checks include: + # - Unused imports/variables + # - Syntax errors + # - Code style issues + # - Potential bugs + # ----------------------- # Python import sorting (compatible with black) # ----------------------- - repo: https://github.com/pycqa/isort @@ -30,7 +45,7 @@ repos: args: ["--profile=black"] # Use black-compatible configuration # Auto-fix import sorting: # 1. Standard library imports - # 2. Third-party library imports + # 2. Third-party library imports # 3. Local application/library specific imports # Separated by blank lines between groups @@ -48,21 +63,6 @@ repos: # - Fast execution # - Produces uniform code style - # ----------------------- - # Python Linting (Ruff) - # Characteristics: Fast, suitable for CI, replaces flake8/isort/etc. - # ----------------------- - - repo: https://github.com/astral-sh/ruff-pre-commit - rev: v0.14.14 # Ruff version - hooks: - - id: ruff-check # Main linting checks - types_or: [ python, pyi ] # Check .py and .pyi files - args: [ --fix ] # Auto-fix fixable issues - # Checks include: - # - Unused imports/variables - # - Syntax errors - # - Code style issues - # - Potential bugs - repo: https://github.com/jackdewinter/pymarkdown rev: v0.9.29 hooks: