Merge pull request #190 from mrbermell/annotations-and-repr

Annotations according to PEP 484 in game.py and model.py

botbowl/core/forward_model.py

@@ -7,12 +7,13 @@
 """
 from copy import deepcopy, copy
 from enum import Enum
-from pytest import set_trace
 
 
 class Reversible:
 
-    def __init__(self, ignored_keys=[]):
+    def __init__(self, ignored_keys=None):
+        if ignored_keys is None:
+            ignored_keys = []
         super().__setattr__("_trajectory", None)
         super().__setattr__("_ignored_keys", set(ignored_keys))
 
@@ -89,7 +90,6 @@ def next_step(self):
             self.current_step += 1
 
 
-
 class Step:
     def undo(self):
         raise NotImplementedError("Method to be overwritten by subclass")
@@ -332,6 +332,12 @@ def is_immutable(obj):
 immutable_types = {int, float, str, tuple, bool, range, type(None)}
 
 
+def treat_as_immutable(cls):
+    """Used as decorator for classes that should never be tracked by forward model"""
+    immutable_types.add(cls)
+    return cls
+
+
 def add_reversibility(value, trajectory):
     if is_immutable(value):
         return value