Skip to content
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

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

Open
pcworld wants to merge 1 commit into
jGleitz: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 into
jGleitz:masterfrom
pcworld:final2-createdlazyant-movement

Conversation

@pcworld

@pcworld pcworld commented Mar 29, 2015

Copy link
Copy Markdown
Collaborator

Adds a simple test for #184

Review on Reviewable

Copy link
Copy Markdown
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

pcworld commented Mar 29, 2015

Copy link
Copy Markdown
Collaborator Author

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

@jGleitz

jGleitz commented Mar 29, 2015

Copy link
Copy Markdown
Owner

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

CommandInteractionTest ?

@pcworld

pcworld commented Mar 29, 2015

Copy link
Copy Markdown
Collaborator Author

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

jGleitz commented Mar 29, 2015

Copy link
Copy Markdown
Owner

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

@jGleitz

jGleitz commented Mar 29, 2015

Copy link
Copy Markdown
Owner

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 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