Skip to content

final2: movement of lazy ants created in between other lazy ants' moves#185

Open
pcworld wants to merge 1 commit intojGleitz:masterfrom
pcworld:final2-createdlazyant-movement
Open

final2: movement of lazy ants created in between other lazy ants' moves#185
pcworld wants to merge 1 commit intojGleitz:masterfrom
pcworld:final2-createdlazyant-movement

Conversation

@pcworld
Copy link
Collaborator

@pcworld pcworld commented Mar 29, 2015

Adds a simple test for #184

Review on Reviewable

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't know why my formatter keeps doing this, or which version is correct according to our formatter file…

@pcworld
Copy link
Collaborator Author

pcworld commented Mar 29, 2015

I couldn't decide whether this belongs in CreateTest or MoveTest.

@jGleitz
Copy link
Owner

jGleitz commented Mar 29, 2015

I couldn't decide whether this belongs in CreateTest or MoveTest.

CommandInteractionTest ?

@pcworld
Copy link
Collaborator Author

pcworld commented Mar 29, 2015

CommandInteractionTest

I don't know. Actually it really tests the move behavior, but in a scenario that can't be created without the create command.

@jGleitz
Copy link
Owner

jGleitz commented Mar 29, 2015

Just leave them where they are. It's not that important, I guess.

@jGleitz
Copy link
Owner

jGleitz commented Mar 29, 2015

As pointed out in the associated ILIAS-Post, your interpretation is not the only one that's valid. I read the description again and came to the conclusion it's ambivalent. Therefore I don't think we should merge it right now until we have an official statement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants