Change 'class-definitions-order' behavior regarding static class variables, fixes #331

Scony · Scony · commit 31ae4f465da7 · 2024-10-07T22:28:03.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@
  - Changed formatting of some uni-statement lambdas
  - Changed formatting of multi-statement, inline lambdas
  - Changed formatting of dot-chains containing a lambda(s)
+ - Changed linter check `class-definitions-order` in a way that static class variables are recommended to be placed just after constants
 
 ## [4.3.1] 2024-08-24
 
diff --git a/gdtoolkit/linter/__init__.py b/gdtoolkit/linter/__init__.py
@@ -64,12 +64,12 @@
             "signals",
             "enums",
             "consts",
+            "staticvars",
             "exports",
             "pubvars",
             "prvvars",
             "onreadypubvars",
             "onreadyprvvars",
-            "staticvars",
             "others",
         ],
         # useless-super-delegation
diff --git a/tests/linter/test_class_checks.py b/tests/linter/test_class_checks.py
@@ -13,6 +13,7 @@
 signal s
 enum { A, B, C }
 const X = 1
+static var sx
 @export_group("Foo")
 @export var k = 1
 var x = 1
