From 35806576f9b3d69f6401069120d327a0b3135b43 Mon Sep 17 00:00:00 2001 From: Marc Di Luzio Date: Sun, 7 Jun 2020 23:17:56 +0100 Subject: [PATCH] Add a bunch of TODOs --- cmd/rove/main.go | 3 +++ pkg/game/math.go | 2 ++ pkg/game/tile.go | 1 + pkg/game/world.go | 1 + pkg/rove/api.go | 2 ++ 5 files changed, 9 insertions(+) diff --git a/cmd/rove/main.go b/cmd/rove/main.go index 2e46736..2fc436a 100644 --- a/cmd/rove/main.go +++ b/cmd/rove/main.go @@ -16,6 +16,7 @@ import ( var USAGE = "" // Command usage +// TODO: Allow COMMAND to be used first func Usage() { fmt.Printf("Usage: %s [OPTIONS]... COMMAND\n", os.Args[0]) fmt.Println("\nCommands:") @@ -173,6 +174,8 @@ func InnerMain(command string) error { } else { fmt.Printf("radar blips: %+v\n", response.Blips) + + // TODO: Do some art } case "rover": diff --git a/pkg/game/math.go b/pkg/game/math.go index 3c9c1b2..059e4ad 100644 --- a/pkg/game/math.go +++ b/pkg/game/math.go @@ -6,6 +6,8 @@ import ( "strings" ) +// TODO: Pull this out into math package and get more test coverage + // Abs gets the absolute value of an int func Abs(x int) int { if x < 0 { diff --git a/pkg/game/tile.go b/pkg/game/tile.go index 954f571..f7d7039 100644 --- a/pkg/game/tile.go +++ b/pkg/game/tile.go @@ -7,6 +7,7 @@ const ( TileEmpty = Tile(0) TileRover = Tile(1) + // TODO: Is there even a difference between these two? TileWall = Tile(2) TileRock = Tile(3) ) diff --git a/pkg/game/world.go b/pkg/game/world.go index 7736551..8bd34e6 100644 --- a/pkg/game/world.go +++ b/pkg/game/world.go @@ -55,6 +55,7 @@ func (w *World) SpawnRover() (uuid.UUID, error) { Range: 5.0, // Set the name randomly + // TODO: Fix the stupid "'s" Name: babble.NewBabbler().Babble(), }, } diff --git a/pkg/rove/api.go b/pkg/rove/api.go index 0163a01..b878111 100644 --- a/pkg/rove/api.go +++ b/pkg/rove/api.go @@ -21,6 +21,8 @@ type StatusResponse struct { Version string `json:"version"` Tick int `json:"tick"` NextTick string `json:"nexttick,omitempty"` + + // TODO: return more useful info } // ==============================