From 93ca90558aa6016e77625c627a70640e693e3303 Mon Sep 17 00:00:00 2001 From: blasty Date: Fri, 21 Aug 2026 12:14:27 +0200 Subject: adopt ruff: pinned formatter + import sorting, opt-in pre-commit hook PR #1 arrived black-formatted, which forced the question. ruff's formatter is black's output, it is one static binary rather than a dependency chain, and it is fast enough that a hook is free. ruff.toml pins the version (a formatter whose output moves between releases turns 'formatted' into 'formatted by whoever committed last') and selects only import sorting on the lint side -- a hook that refused a commit over an unused variable would be a different feature. .githooks/pre-commit (install: git config core.hooksPath .githooks) formats exactly what is staged, refuses partly-staged files rather than quietly widening a commit built with git add -p, and prefers the pinned .venv ruff. Also drops the blanket '# ruff: noqa' PR #1 left in remote_ops.py -- it would have exempted that file from the only lint rule we enforce. --- idatui/remote_ops.py | 1 - 1 file changed, 1 deletion(-) (limited to 'idatui') diff --git a/idatui/remote_ops.py b/idatui/remote_ops.py index cbd1d49..b07bb71 100644 --- a/idatui/remote_ops.py +++ b/idatui/remote_ops.py @@ -1,7 +1,6 @@ """Typed remote operations executed through ida-nexus.""" from __future__ import annotations -# ruff: noqa import threading from collections.abc import Callable -- cgit v1.3.1-sl0p