Merge pull request #1695 from Kamoppl/kamilg/pep8_fix

github-actions: update code style checker for python

Author: robertbaldyga

.github/workflows/ruff.yml

@@ -0,0 +1,18 @@
+name: ruff review
+on: [pull_request]
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Install Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.11"
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install ruff
+      # Update output format to enable automatic inline annotations.
+      - name: Run Ruff
+        run: ruff check --config .ruff.toml --exclude "test/functional/test-framework/**/*.py" --exclude "doc/*" --exclude "doc/*" --output-format=github .

.pep8speaks.yml

@@ -0,0 +1,8 @@
+line-length = 100
+
+[lint]
+ignore = [
+    "E402", # All module level imports should be at the top of the file. This means that there
+            # should be no statements in between module level imports
+    "F405", # Variable  may be undefined, or defined from star imports
+]

.ruff.toml

@@ -16,7 +16,7 @@
     PromotionPolicy,
 )
 from api.cas.casadm_params import OutputFormat, StatsFilter
-from api.cas.cli import *
+from api.cas.cli import * # noqa: F403
 from api.cas.core import Core
 from core.test_run import TestRun
 from storage_devices.device import Device

test/functional/api/cas/casadm.py

@@ -12,8 +12,8 @@
 from typing import List
 
 from api.cas import casadm
-from api.cas.cache_config import *
-from api.cas.casadm_params import *
+from api.cas.cache_config import * # noqa: F403
+from api.cas.casadm_params import * # noqa: F403
 from api.cas.core_config import CoreStatus
 from api.cas.ioclass_config import IoClass
 from api.cas.version import CasVersion

test/functional/api/cas/casadm_parser.py

@@ -1,9 +1,10 @@
 #
 # Copyright(c) 2019-2021 Intel Corporation
+# Copyright(c) 2025 Huawei Technologies Co., Ltd.
 # SPDX-License-Identifier: BSD-3-Clause
 #
 
-from .cli import *
+from .cli import * # noqa: F403
 from core.test_run import TestRun
 
 

test/functional/api/cas/casctl.py

@@ -6,7 +6,7 @@
 
 import pytest
 
-from api.cas.cli_help_messages import *
+from api.cas.cli_help_messages import * # noqa: F403
 from api.cas.cli_messages import check_stderr_msg, check_stdout_msg
 from core.test_run import TestRun
 

test/functional/tests/cli/test_cli_help.py

@@ -13,7 +13,6 @@
 from type_def.size import Size, Unit
 from api.cas.cache_config import CacheLineSize, CacheMode
 from api.cas.cli import standby_activate_cmd, standby_load_cmd
-from api.cas.dmesg import get_md_section_size
 from test_tools.dd import Dd
 from test_tools.os_tools import sync
 from test_utils.filesystem.file import File

test/functional/tests/fault_injection/test_fault_injection_standby.py

@@ -11,7 +11,7 @@
 
 from storage_devices.disk import DiskType, DiskTypeSet, DiskTypeLowerThan
 from test_tools.fs_tools import Filesystem, create_directory
-from .io_class_common import *
+from .io_class_common import * # noqa: F403
 
 
 @pytest.mark.require_disk("cache", DiskTypeSet([DiskType.optane, DiskType.nand]))

test/functional/tests/io_class/test_io_class_eviction_priority.py

@@ -27,7 +27,6 @@
     get_basic_workload,
     mount_point,
 )
-from type_def.size import Unit
 
 dirty_ratio_inertia_min, dirty_ratio_inertia_max = FlushParametersAlru.alru_params_range().dirty_ratio_inertia