From e254fec310dac598b6ab69d3abc57a4d43d56e8a Mon Sep 17 00:00:00 2001 From: qlrd Date: Thu, 26 Mar 2026 14:49:40 -0300 Subject: [PATCH 1/2] chore: open review for docs/claude-md From ed53f700d8217a3058f3f44b86584d9e08df2126 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 26 Mar 2026 18:35:29 +0000 Subject: [PATCH 2/2] docs: clarify Black vs Pylint line length and pytest runner usage Agent-Logs-Url: https://github.com/qlrd/krux-installer/sessions/44256ca2-7e45-4816-b6e6-4ce029b6d28f Co-authored-by: qlrd <106913782+qlrd@users.noreply.github.com> --- CLAUDE.md | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 63da1f67..a87729fa 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -52,8 +52,9 @@ and pass tests. ## Code style -- **Formatter:** Black (max line length follows Black defaults) -- **Linter:** Pylint (max line: 100, max statements: 50, see `.pylint/src`) +- **Formatter:** Black (enforces 88-char line length for all code) +- **Linter:** Pylint (max line: 100 as a lenient upper bound for non-Black-managed cases, + max statements: 50, see `.pylint/src`) - **Python:** >=3.12.0 required - **Naming:** `snake_case` (functions, variables), `PascalCase` (classes), `UPPER_CASE` (constants) @@ -100,7 +101,7 @@ To add a new device: ## Testing -- Framework: `pytest` with `unittest.TestCase` +- **Test runner:** `pytest`; some test classes use `unittest.TestCase`-style structure - Mocking: `unittest.mock` (`patch`, `MagicMock`) - E2E: `kivy.tests.common.GraphicUnitTest` - Shared mocks in `tests/shared_mocks.py`