From 4215d2cd931d37845f7fc68dbb50ecf0c889e401 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Bergstr=C3=B6m?= <davbe125@student.liu.se>
Date: Mon, 6 Aug 2018 11:49:41 +0200
Subject: [PATCH] Update/resolve a few TODOs

---
 docs/gettingstarted.rst | 4 ----
 docs/managers.rst       | 7 ++-----
 docs/unit.rst           | 1 -
 3 files changed, 2 insertions(+), 10 deletions(-)

diff --git a/docs/gettingstarted.rst b/docs/gettingstarted.rst
index c8c6e1f..29a31fc 100644
--- a/docs/gettingstarted.rst
+++ b/docs/gettingstarted.rst
@@ -1,5 +1,3 @@
-.. TODO: How should we host the binaries
-
 .. _gettingstarted:
 
 Getting started
@@ -49,8 +47,6 @@ Now, let us break it down piece by piece to understand it.
 
    from library import *
 
-.. TODO: Should we really encourage this behaviour?
-
 This imports everything from the library into your namespace. 
 
 Next, we need to define our bot.
diff --git a/docs/managers.rst b/docs/managers.rst
index 926ce11..2e861ce 100644
--- a/docs/managers.rst
+++ b/docs/managers.rst
@@ -79,12 +79,8 @@ TypeData
    :members:
    :undoc-members:
 
-.. TODO: Where is morph ability?
-
 .. TODO: Types for all fields would be nice, is required_units List[UnitType] or List[UNIT_TYPEID]?
 
-.. TODO: Not all members have docstrings attached to them
-
 MapTools
 --------
 
@@ -98,7 +94,8 @@ Color
 .. autoclass:: library.Color 	
    :members:
    :undoc-members:
-.. TODO: Only include the constructor and the color constants
+
+   .. automethod:: __init__
 
 DistanceMap
 ~~~~~~~~~~~
diff --git a/docs/unit.rst b/docs/unit.rst
index 0ffe399..003e12d 100644
--- a/docs/unit.rst
+++ b/docs/unit.rst
@@ -58,7 +58,6 @@ Unit
    .. automethod:: right_click
    .. automethod:: repair
    .. automethod:: build
-   .. TODO: Maybe should I put the limits of build here, and when to use build_target
    .. automethod:: build_target
    .. automethod:: train
    .. automethod:: research
-- 
GitLab