Refresh and types

mypy.ini

@@ -1,4 +1,26 @@
 [mypy]
+files =
+    src/**/*.py,
+    tests/**/*.py
+warn_redundant_casts = True
+warn_unused_configs = True
+pretty = True
+show_error_codes = True
+
+disallow_any_generics = True
+disallow_subclassing_any = True
+# disallow_untyped_calls = True
+disallow_incomplete_defs = True
+# check_untyped_defs = True
+disallow_untyped_decorators = True
+no_implicit_optional = True
+warn_unused_ignores = True
+warn_return_any = True
+no_implicit_reexport = True
+
+# More strict checks for library code
+[mypy-czml3]
+disallow_untyped_defs = True
 
 [mypy-versioneer]
 ignore_errors = True

setup.py

@@ -1,4 +1,5 @@
-import versioneer
 from setuptools import setup
 
+import versioneer
+
 setup(version=versioneer.get_version(), cmdclass=versioneer.get_cmdclass())

src/czml3/base.py

@@ -25,7 +25,7 @@ def default(self, o):
         return super().default(o)
 
 
-@attr.s(repr=False, frozen=True)
+@attr.s(auto_attribs=True, repr=False, frozen=True)
 class BaseCZMLObject:
     def __repr__(self):
         return self.dumps(indent=4)

src/czml3/common.py

@@ -1,35 +1,36 @@
 # noinspection PyPep8Naming
 import datetime as dt
+from typing import Optional
 
 import attr
 
 from .enums import HorizontalOrigins, InterpolationAlgorithms, VerticalOrigins
 
 
-@attr.s(repr=False, frozen=True, kw_only=True)
+@attr.s(auto_attribs=True, repr=False, frozen=True, kw_only=True)
 class Deletable:
     """A property whose value may be deleted."""
 
-    delete: bool = attr.ib(default=None)
+    delete: Optional[bool] = None
 
 
 # noinspection PyPep8Naming
-@attr.s(repr=False, frozen=True, kw_only=True)
+@attr.s(auto_attribs=True, repr=False, frozen=True, kw_only=True)
 class Interpolatable:
     """A property whose value may be determined by interpolating.
 
     The interpolation happens over provided time-tagged samples.
     """
 
-    epoch: dt.datetime = attr.ib(default=None)
-    interpolationAlgorithm: InterpolationAlgorithms = attr.ib(default=None)
-    interpolationDegree: int = attr.ib(default=None)
+    epoch: Optional[dt.datetime] = None
+    interpolationAlgorithm: Optional[InterpolationAlgorithms] = None
+    interpolationDegree: Optional[int] = None
 
 
 # noinspection PyPep8Naming
-@attr.s(repr=False, frozen=True, kw_only=True)
+@attr.s(auto_attribs=True, repr=False, frozen=True, kw_only=True)
 class HasAlignment:
     """A property that can be horizontally or vertically aligned."""
 
-    horizontalOrigin: HorizontalOrigins = attr.ib(default=None)
-    verticalOrigin: VerticalOrigins = attr.ib(default=None)
+    horizontalOrigin: Optional[HorizontalOrigins] = None
+    verticalOrigin: Optional[VerticalOrigins] = None