Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Finish removing references to snares. #4285

Merged

Conversation

Projects
None yet
4 participants
@ianestrachan
Copy link
Contributor

commented Nov 11, 2013

Fixes #4275 by removing all remaining references to snares; placing land mines now puts down land mines, and placing caltrops actually places caltrops, etc.

@freezerbunny

This comment has been minimized.

Copy link
Contributor

commented Nov 14, 2013

Yeah this is essential.

@ianestrachan

This comment has been minimized.

Copy link
Contributor Author

commented Nov 15, 2013

Is there something holding this back from being merged? Everyone is seeing it and it's a relatively minor fix. I tried placing all traps and the only one that didn't seem to quite work was the tripwire, which I think I was just using wrong.

@i2amroy

This comment has been minimized.

Copy link
Member

commented Nov 15, 2013

More like Kevin was gone on a trip and I've been a bit busy. I'll take a look into merging it in a bit.

Rivet-the-Zombie added a commit that referenced this pull request Nov 16, 2013

@Rivet-the-Zombie Rivet-the-Zombie merged commit 20d5db0 into CleverRaven:master Nov 16, 2013

1 check passed

default Merged build finished.
Details

@ianestrachan ianestrachan deleted the ianestrachan:totally-remove-snares branch Nov 16, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.