ci: Optimize CI workflow

HzaCode · HzaCode · commit acd018eefd3d · 2025-10-27T20:25:50.000+08:00
- Only run tests on macOS (3.10, 3.11) - fastest and most stable
- Replace black+isort+flake8 with ruff for faster linting and auto-fixing
- Add ruff.toml configuration
- Fix unused variables in conftest.py
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -11,18 +11,8 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
-        python-version: ["3.8", "3.9", "3.10", "3.11"]
-        exclude:
-          # Reduce matrix size, test all Python versions only on Ubuntu
-          - os: windows-latest
-            python-version: "3.8"
-          - os: windows-latest
-            python-version: "3.9"
-          - os: macos-latest
-            python-version: "3.8"
-          - os: macos-latest
-            python-version: "3.9"
+        os: [macos-latest]
+        python-version: ["3.10", "3.11"]
 
     steps:
     - uses: actions/checkout@v4
@@ -76,20 +66,13 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install flake8 black isort mypy
+        pip install ruff mypy
 
-    - name: Lint with flake8
+    - name: Lint and format with ruff
       run: |
-        flake8 huez tests --count --select=E9,F63,F7,F82 --show-source --statistics
-        flake8 huez tests --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-
-    - name: Check formatting with black
-      run: |
-        black --check huez tests
-
-    - name: Check import sorting with isort
-      run: |
-        isort --check-only huez tests
+        # Check and auto-fix with ruff
+        ruff check --fix huez tests
+        ruff format huez tests
 
     - name: Type check with mypy
       run: |
diff --git a/docs/source/api/adapters.rst b/docs/source/api/adapters.rst
@@ -45,3 +45,4 @@ plotnine
 
 
 
+
diff --git a/docs/source/api/utils.rst b/docs/source/api/utils.rst
@@ -31,3 +31,4 @@ Quality Checks
 
 
 
+
diff --git a/docs/source/changelog.rst b/docs/source/changelog.rst
@@ -103,3 +103,4 @@ See :doc:`contributing` to help shape future releases!
 
 
 
+
diff --git a/docs/source/comparison.rst b/docs/source/comparison.rst
@@ -693,3 +693,4 @@ Next Steps
 - See :doc:`faq` for common questions
 
 
+
diff --git a/docs/source/contributing.rst b/docs/source/contributing.rst
@@ -165,3 +165,4 @@ Every contribution, no matter how small, is appreciated! 🎉
 
 
 
+
diff --git a/docs/source/faq.rst b/docs/source/faq.rst
@@ -637,3 +637,4 @@ Still Have Questions?
 - Open an issue: https://github.com/hzacode/huez/issues
 
 
+
diff --git a/docs/source/gallery/comparisons.rst b/docs/source/gallery/comparisons.rst
@@ -132,3 +132,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/gallery/index.rst b/docs/source/gallery/index.rst
@@ -54,3 +54,4 @@ All palettes available via ``hz.use("palette_name")``.
 
 
 
+
diff --git a/docs/source/gallery/palettes.rst b/docs/source/gallery/palettes.rst
@@ -150,3 +150,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/intelligence/chart_adaptation.rst b/docs/source/intelligence/chart_adaptation.rst
@@ -426,3 +426,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/library_support/altair.rst b/docs/source/library_support/altair.rst
@@ -51,3 +51,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/library_support/index.rst b/docs/source/library_support/index.rst
@@ -219,3 +219,4 @@ Explore detailed documentation for each library:
 
 
 
+
diff --git a/docs/source/library_support/plotly.rst b/docs/source/library_support/plotly.rst
@@ -46,3 +46,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/library_support/plotnine.rst b/docs/source/library_support/plotnine.rst
@@ -58,3 +58,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/library_support/seaborn.rst b/docs/source/library_support/seaborn.rst
@@ -57,3 +57,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/user_guide/best_practices.rst b/docs/source/user_guide/best_practices.rst
@@ -472,3 +472,4 @@ Next Steps
 
 
 
+
diff --git a/docs/source/why_huez.rst b/docs/source/why_huez.rst
@@ -705,3 +705,4 @@ Join the Community
 **Your time is valuable. Spend it on science, not color configuration.**
 
 
+
diff --git a/huez/core.py b/huez/core.py
@@ -105,7 +105,7 @@ def load_config(path: Optional[str] = None) -> Config:
 
         _current_config = get_default_config()
     else:
-        with open(path, "r", encoding="utf-8") as f:
+        with open(path, encoding="utf-8") as f:
             data = yaml.safe_load(f)
         _current_config = Config.from_dict(data)
 
@@ -854,7 +854,7 @@ def preview(scheme_name: Optional[str] = None, mode: str = "screen") -> None:
     x = np.linspace(0, 10, 100)
     for i in range(min(5, len(colors))):
         y = np.sin(x + i * np.pi / 4) + i
-        ax2.plot(x, y, color=colors[i], linewidth=2, label=f"Series {i+1}")
+        ax2.plot(x, y, color=colors[i], linewidth=2, label=f"Series {i + 1}")
     ax2.legend(fontsize=8, loc="best")
     ax2.grid(True, alpha=0.3)
 
diff --git a/ruff.toml b/ruff.toml
@@ -0,0 +1,78 @@
+# Ruff configuration for Huez
+# https://docs.astral.sh/ruff/
+
+# Exclude common directories
+exclude = [
+    ".bzr",
+    ".direnv",
+    ".eggs",
+    ".git",
+    ".hg",
+    ".mypy_cache",
+    ".nox",
+    ".pants.d",
+    ".pytype",
+    ".ruff_cache",
+    ".svn",
+    ".tox",
+    ".venv",
+    "__pypackages__",
+    "_build",
+    "buck-out",
+    "build",
+    "dist",
+    "node_modules",
+    "venv",
+]
+
+# Line length to match black
+line-length = 88
+
+# Target Python 3.8+
+target-version = "py38"
+
+[lint]
+# Enable pycodestyle (E, W), Pyflakes (F), isort (I), and more
+select = [
+    "E",   # pycodestyle errors
+    "W",   # pycodestyle warnings
+    "F",   # Pyflakes
+    "I",   # isort
+    "B",   # flake8-bugbear
+    "C4",  # flake8-comprehensions
+    "UP",  # pyupgrade
+]
+
+# Ignore some rules
+ignore = [
+    "E501",  # line too long (handled by formatter)
+    "B008",  # do not perform function calls in argument defaults
+    "C901",  # too complex
+]
+
+# Allow autofix for all enabled rules
+fixable = ["ALL"]
+unfixable = []
+
+# Allow unused variables when underscore-prefixed
+dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
+
+[lint.per-file-ignores]
+# Ignore import order in __init__.py files
+"__init__.py" = ["F401", "E402"]
+# Test files can use assert and magic numbers
+"tests/**/*.py" = ["S101", "PLR2004"]
+
+[format]
+# Use double quotes for strings
+quote-style = "double"
+
+# Indent with spaces
+indent-style = "space"
+
+# Respect magic trailing commas
+skip-magic-trailing-comma = false
+
+# Auto-detect line endings
+line-ending = "auto"
+
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -3,8 +3,8 @@
 """
 
 import os
-import pytest
 
+import pytest
 
 # Module-level config to ensure persistence
 _test_config = None
@@ -29,12 +29,9 @@ def pytest_configure(config):
 @pytest.fixture(autouse=True)
 def ensure_test_config():
     """Ensure test configuration is loaded for each test."""
-    import huez.core as core
     import matplotlib.pyplot as plt
 
-    # Save original state
-    original_config = core._current_config
-    original_scheme = core._current_scheme
+    import huez.core as core
 
     # Ensure config is loaded
     if _test_config is not None:
diff --git a/tests/test_e2e.py b/tests/test_e2e.py
@@ -447,3 +447,4 @@ def test_advanced_user_workflow(self):
 
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])
+
diff --git a/tests/unit/test_core_detailed.py b/tests/unit/test_core_detailed.py
@@ -554,3 +554,4 @@ def test_setup_alias(self):
 
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])
+

Original file line number	Diff line number	Diff line change
`@@ -31,3 +31,4 @@ Quality Checks`
`31`	`31`
`32`	`32`
`33`	`33`
	`34`	`+`
Original file line number	Diff line number	Diff line change
@@ -103,3 +103,4 @@ See :doc:`contributing` to help shape future releases!
`103`	`103`
`104`	`104`
`105`	`105`
	`106`	`+`
Original file line number	Diff line number	Diff line change
`@@ -693,3 +693,4 @@ Next Steps`
`693`	`693`	- See :doc:`faq` for common questions
`694`	`694`
`695`	`695`
	`696`	`+`
Original file line number	Diff line number	Diff line change
`@@ -165,3 +165,4 @@ Every contribution, no matter how small, is appreciated! 🎉`
`165`	`165`
`166`	`166`
`167`	`167`
	`168`	`+`
Original file line number	Diff line number	Diff line change
`@@ -637,3 +637,4 @@ Still Have Questions?`
`637`	`637`	`- Open an issue: https://github.com/hzacode/huez/issues`
`638`	`638`
`639`	`639`
	`640`	`+`
Original file line number	Diff line number	Diff line change
`@@ -132,3 +132,4 @@ Next Steps`
`132`	`132`
`133`	`133`
`134`	`134`
	`135`	`+`
Original file line number	Diff line number	Diff line change
@@ -54,3 +54,4 @@ All palettes available via ``hz.use("palette_name")``.
`54`	`54`
`55`	`55`
`56`	`56`
	`57`	`+`
Original file line number	Diff line number	Diff line change
`@@ -150,3 +150,4 @@ Next Steps`
`150`	`150`
`151`	`151`
`152`	`152`
	`153`	`+`