2 mistakes in level description fixed #41

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@senk
senk commented Dec 21, 2011

Fixed the irritating mistake in README file for beginner/level2. changed 2 times from ".empty?" to ".enemy?"

@gbataille

I would argue that for level 2, this is good enough. If there is something, smash it, otherwise, move along.
It gets more tricky when the captives get introduced later and you have to know what occupies the tile.
Now it's true that enemy? does not get introduced anywhere else, but it seems to be equivalent to !empty? and captive?

Anyway, looks mostly like a matter of taste :)
I'll keep my empty? implementation for now on my side.

@senk
senk commented Feb 26, 2012

But then we should correct the incoherence with the README.rdoc at "Here is a simple example which will instruct the warrior to attack if he feels an enemy, otherwise he will walk forward."

@gbataille

ok right, works for me :)

@senk senk closed this Feb 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment