Skip to content
This repository has been archived by the owner on Apr 17, 2018. It is now read-only.

Added equipment tracking #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Added equipment tracking #4

wants to merge 3 commits into from

Conversation

HDeffo
Copy link

@HDeffo HDeffo commented Feb 9, 2016

knowing what is equipped will help a lot with automation.

itll be very helpful if turtles can remember what they have equipped. Sorry if i didn't write this well enough according to how you have LAMA written I only briefly read through in order to add these
Added equipment tracking
@@ -21,6 +21,8 @@ local env = getfenv()
local fuel = {}
local facing = {}
local position = {}
local leftEquipment = {}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't use left/right, just use equipment

@dirthsj
Copy link
Owner

dirthsj commented Feb 9, 2016

I won't accept this because it won't work (currently). Also, the correct way for this to be implemented would be to add an equipment table, and equipment.left / equipment.right as whatever is currently equipped.

thats what you get for programming at 3 in the morning ._.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants