[ci] use Ruff linter instead of isort (#6755)

* Update append-comment.sh * Update static_analysis.yml * Update static_analysis.yml * Update basic.py * Update basic.py * Update .pre-commit-config.yaml * Update basic.py * Update basic.py * Update basic.py * Update basic.py * Update basic.py * Update pyproject.toml * Update pyproject.toml * Update pyproject.toml * Update pyproject.toml * Update interactive_plot_example.ipynb * Update pyproject.toml * Update append-comment.sh * Update basic.py * Update basic.py * Update pyproject.toml * Update .pre-commit-config.yaml * Update basic.py * Update basic.py * Update test_basic.R * Update rank_objective.hpp * Update histogram_16_64_256.cu * Update static_analysis.yml * ensure alphabetical order of rules
microsoft · Dec 15, 2024 · c2f3807 · c2f3807
1 parent 1090a93
commit c2f3807
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 30 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -17,15 +17,9 @@ repos:
     hooks:
       - id: end-of-file-fixer
       - id: trailing-whitespace
-  - repo: https://github.com/pycqa/isort
-    rev: 5.13.2
-    hooks:
-      - id: isort
-        name: isort (python)
-        args: ["--settings-path", "python-package/pyproject.toml"]
   - repo: https://github.com/astral-sh/ruff-pre-commit
     # Ruff version.
-    rev: v0.7.0
+    rev: v0.8.3
     hooks:
       # Run the linter.
       - id: ruff
@@ -40,7 +34,7 @@ repos:
     hooks:
       - id: shellcheck
   - repo: https://github.com/crate-ci/typos
-    rev: v1.23.2
+    rev: v1.28.3
     hooks:
       - id: typos
         args: ["--force-exclude"]

diff --git a/R-package/tests/testthat/test_basic.R b/R-package/tests/testthat/test_basic.R
@@ -2345,7 +2345,7 @@ test_that("early stopping works with lgb.cv()", {
   # never changes, its first iteration was the best oone
   expect_equal(bst$best_iter, 1L)
 
-  # best_score should be taken from the first metri
+  # best_score should be taken from the first metric
   expect_equal(bst$best_score, 0.2)
 
   # early stopping should have happened, since constant_metric was the first

diff --git a/examples/python-guide/notebooks/interactive_plot_example.ipynb b/examples/python-guide/notebooks/interactive_plot_example.ipynb
@@ -30,7 +30,7 @@
     "try:\n",
     "    # To enable interactive mode you should install ipywidgets\n",
     "    # https://github.com/jupyter-widgets/ipywidgets\n",
-    "    from ipywidgets import interact, SelectMultiple\n",
+    "    from ipywidgets import SelectMultiple, interact\n",
     "\n",
     "    INTERACTIVE = True\n",
     "except ImportError:\n",

diff --git a/python-package/lightgbm/basic.py b/python-package/lightgbm/basic.py
@@ -2504,13 +2504,13 @@ def _compare_params_for_warning(
         compare_result : bool
           Returns whether two dictionaries with params are equal.
         """
-        for k in other_params:
+        for k, v in other_params.items():
             if k not in ignore_keys:
-                if k not in params or params[k] != other_params[k]:
+                if k not in params or params[k] != v:
                     return False
-        for k in params:
+        for k, v in params.items():
             if k not in ignore_keys:
-                if k not in other_params or params[k] != other_params[k]:
+                if k not in other_params or v != other_params[k]:
                     return False
         return True
 

diff --git a/python-package/pyproject.toml b/python-package/pyproject.toml
@@ -84,17 +84,6 @@ minimum-version = "build-system.requires"
 
 # end:build-system
 
-[tool.isort]
-include_trailing_comma = true
-line_length = 120
-# "vertical hanging indent", to match what ruff-format does
-# ref: https://pycqa.github.io/isort/docs/configuration/multi_line_output_modes.html#3-vertical-hanging-indent
-multi_line_output = 3
-skip_glob = [
-    "*/external_libs/*",
-    "*/lightgbm-python/*",
-]
-
 [tool.mypy]
 disallow_untyped_defs = true
 exclude = 'build/*|compile/*|docs/*|examples/*|external_libs/*|lightgbm-python/*|tests/*'
@@ -140,7 +129,7 @@ ignore = [
     "PLR1714",
     # (pylint) Magic value used in comparison
     "PLR2004",
-    # (pylint) for loop veriable overwritten by assignment target
+    # (pylint) for loop variable overwritten by assignment target
     "PLW2901",
     # (pylint) use 'elif' instead of 'else' then 'if', to reduce indentation
     "PLR5501"
@@ -152,10 +141,12 @@ select = [
     "C4",
     # pydocstyle
     "D",
-    # pycodestyle
+    # pycodestyle (errors)
     "E",
     # pyflakes
     "F",
+    # isort
+    "I",
     # NumPy-specific rules
     "NPY",
     # pylint
@@ -166,11 +157,13 @@ select = [
     "SIM401",
     # flake8-print
     "T",
+    # pycodestyle (warnings)
+    "W",
 ]
 
 [tool.ruff.lint.per-file-ignores]
 "docs/conf.py" = [
-    # (flake8-bugbear) raise exceptions with "raise ... from errr"
+    # (flake8-bugbear) raise exceptions with "raise ... from err"
     "B904",
     # (flake8-print) flake8-print
     "T"
@@ -196,3 +189,6 @@ select = [
 
 [tool.ruff.lint.pydocstyle]
 convention = "numpy"
+
+[tool.ruff.lint.isort]
+known-first-party = ["lightgbm"]
diff --git a/src/objective/rank_objective.hpp b/src/objective/rank_objective.hpp
@@ -204,7 +204,7 @@ class LambdarankNDCG : public RankingObjective {
     }
     const double worst_score = score[sorted_idx[worst_idx]];
     double sum_lambdas = 0.0;
-    // start accmulate lambdas by pairs that contain at least one document above truncation level
+    // start accumulate lambdas by pairs that contain at least one document above truncation level
     for (data_size_t i = 0; i < cnt - 1 && i < truncation_level_; ++i) {
       if (score[sorted_idx[i]] == kMinScore) { continue; }
       for (data_size_t j = i + 1; j < cnt; ++j) {

diff --git a/src/treelearner/kernels/histogram_16_64_256.cu b/src/treelearner/kernels/histogram_16_64_256.cu
@@ -150,7 +150,7 @@ __global__ void KERNEL_NAME(const uchar* feature_data_base,
     // size of threads that process this feature4
     const unsigned int subglobal_size = lsize * (1 << power_feature_workgroups);
 
-    // equavalent thread ID in this subgroup for this feature4
+    // equivalent thread ID in this subgroup for this feature4
     const unsigned int subglobal_tid  = gtid - feature_id * subglobal_size;