Merge pull request #217 from nucleic/ruff-fixes

Sort slots and __all__ attrs (fix linting issues)

Author: MatthieuDartiailh

atom/api.py

@@ -67,58 +67,58 @@
 __all__ = [
     "Atom",
     "AtomMeta",
-    "MissingMemberWarning",
-    "add_member",
-    "observe",
-    "set_default",
-    "clone_if_needed",
+    "Bool",
+    "Bytes",
     "CAtom",
+    "Callable",
+    "ChangeDict",
     "ChangeType",
-    "DefaultValue",
-    "GetAttr",
-    "GetState",
-    "Member",
-    "PostGetAttr",
-    "PostSetAttr",
-    "PostValidate",
-    "SetAttr",
-    "Validate",
-    "atomclist",
-    "atomdict",
-    "defaultatomdict",
-    "atomlist",
-    "atomref",
-    "atomset",
     "Coerced",
+    "Constant",
     "ContainerList",
+    "DefaultDict",
+    "DefaultValue",
     "Delegator",
     "Dict",
-    "DefaultDict",
     "Enum",
     "Event",
+    "FixedTuple",
+    "Float",
+    "FloatRange",
     "ForwardInstance",
+    "ForwardSubclass",
+    "ForwardTyped",
+    "GetAttr",
+    "GetState",
     "Instance",
+    "Int",
     "List",
+    "Member",
+    "MissingMemberWarning",
+    "PostGetAttr",
+    "PostSetAttr",
+    "PostValidate",
     "Property",
-    "cached_property",
-    "Bool",
-    "Bytes",
-    "Callable",
-    "Constant",
-    "Float",
-    "FloatRange",
-    "Int",
     "Range",
     "ReadOnly",
-    "Str",
-    "Value",
     "Set",
+    "SetAttr",
     "Signal",
-    "ForwardSubclass",
+    "Str",
     "Subclass",
     "Tuple",
-    "ForwardTyped",
     "Typed",
-    "FixedTuple",
-    "ChangeDict",
+    "Validate",
+    "Value",
+    "add_member",
+    "atomclist",
+    "atomdict",
+    "atomlist",
+    "atomref",
+    "atomset",
+    "cached_property",
+    "clone_if_needed",
+    "defaultatomdict",
+    "observe",
+    "set_default",
 ]

atom/instance.py

@@ -99,7 +99,7 @@ class ForwardInstance(Instance):
 
     """
 
-    __slots__ = ("resolve", "args", "kwargs", "optional")
+    __slots__ = ("args", "kwargs", "optional", "resolve")
 
     def __init__(self, resolve, args=None, kwargs=None, *, factory=None, optional=None):
         """Initialize a ForwardInstance.

atom/meta/__init__.py

@@ -13,9 +13,9 @@
 
 __all__ = [
     "AtomMeta",
-    "observe",
     "MissingMemberWarning",
     "add_member",
-    "set_default",
     "clone_if_needed",
+    "observe",
+    "set_default",
 ]

atom/meta/atom_meta.py

@@ -202,22 +202,22 @@ class _AtomMetaHelper:
     getstates: List[str]
 
     __slots__ = (
-        "name",
         "bases",
         "dct",
+        "decorated",
+        "defaults",
+        "getstates",
         "members",
-        "owned_members",
-        "specific_members",
+        "name",
         "observes",
-        "defaults",
-        "validates",
-        "decorated",
-        "set_defaults",
+        "owned_members",
         "post_getattrs",
         "post_setattrs",
         "post_validates",
-        "getstates",
         "seen_decorated",
+        "set_defaults",
+        "specific_members",
+        "validates",
     )
 
     def __init__(self, name: str, bases: Tuple[type, ...], dct: Dict[str, Any]) -> None:

atom/meta/member_modifiers.py

@@ -13,7 +13,7 @@
 class set_default(object):
     """An object used to set the default value of a base class member."""
 
-    __slots__ = ("value", "name")
+    __slots__ = ("name", "value")
 
     #: Name of the member for which a new default value should be set. Used by
     #: the metaclass.

atom/meta/observation.py

@@ -65,7 +65,7 @@ def observe(*names: str, change_types: ChangeType = ChangeType.ANY) -> "ObserveH
 class ObserveHandler(object):
     """An object used to temporarily store observe decorator state."""
 
-    __slots__ = ("pairs", "func", "funcname", "change_types")
+    __slots__ = ("change_types", "func", "funcname", "pairs")
 
     #: List of 2-tuples which stores the pair information for the observers.
     pairs: List[Tuple[str, Optional[str]]]
@@ -132,7 +132,7 @@ def clone(self) -> "ObserveHandler":
 class ExtendedObserver(object):
     """A callable object used to implement extended observers."""
 
-    __slots__ = ("funcname", "attr")
+    __slots__ = ("attr", "funcname")
 
     #: Name of the function on the owner object which should be used as the observer.
     funcname: str

atom/typed.py

@@ -99,7 +99,7 @@ class ForwardTyped(Typed):
 
     """
 
-    __slots__ = ("resolve", "args", "kwargs", "optional")
+    __slots__ = ("args", "kwargs", "optional", "resolve")
 
     def __init__(self, resolve, args=None, kwargs=None, *, factory=None, optional=None):
         """Initialize a ForwardTyped.

setup.py

@@ -5,7 +5,6 @@
 #
 # The full license is in the file LICENSE, distributed with this software.
 # --------------------------------------------------------------------------------------
-import os
 
 from setuptools import Extension, setup
 

tests/test_get_set_state.py

@@ -227,7 +227,7 @@ def task():
 @pytest.mark.parametrize("fn", ("c", "py"))
 def test_bench_getstate_slots(benchmark, fn):
     class Test(AtomBase):
-        __slots__ = ("foo", "bar")
+        __slots__ = ("bar", "foo")
         name = Str("Name")
         enabled = Bool(True)
         rating = Float()

tests/type_checking/test_annotations.yml

@@ -13,7 +13,7 @@
       member_type: atom.list.List[builtins.int]
       member_value_type: builtins.list[builtins.int]
   main: |
-    import io
+    import _io
     from typing import Type
     from atom.api import Atom, {{ member }}