From b2b43f71440b4ac36eb27e0b79c070f78665c029 Mon Sep 17 00:00:00 2001
From: Tim Jaacks <tim.jaacks@seco.com>
Date: Mon, 28 Aug 2023 15:14:06 +0200
Subject: [PATCH] Sourcery: keep "use-named-expression" check disabled

This suggestion aims to reduce code size, but IMO for the expense of
readability for non-python-experts, so we disable it.

See for reference:
https://docs.sourcery.ai/Reference/Python/Default-Rules/use-named-expression/
---
 scripts/.sourcery.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/.sourcery.yaml b/scripts/.sourcery.yaml
index e84ee339..3a3eb0f8 100644
--- a/scripts/.sourcery.yaml
+++ b/scripts/.sourcery.yaml
@@ -10,13 +10,14 @@ rule_settings:
   - min-max-identity
   # Keep explicit lower range limit instead of using implicit default value
   - remove-zero-from-range
+  # Don't combine two operations on a single line
+  - use-named-expression
   # Requires special Python knowlege that not everyone reading the code might have
   - use-next
   # FIXME: verify if we want to keep the checks below.
   # If not, remove them. If yes, move them above this comment.
   - low-code-quality
   - for-append-to-extend
-  - use-named-expression
 
   rule_types:
   - refactoring
-- 
GitLab