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

.bomb wire fix #8

Closed
tyrope opened this issue Jan 9, 2012 · 1 comment
Closed

.bomb wire fix #8

tyrope opened this issue Jan 9, 2012 · 1 comment
Assignees
Labels
Bug Things to squish; generally used for issues

Comments

@tyrope
Copy link

tyrope commented Jan 9, 2012

[0242.54] .cutwire blue
[0242.54] <&Willie> No! No, that's the wrong one. Aww, you've gone and killed yourself. Oh, that's... that's not good. No good at all, really. Wow. Sorry.
[0242.54] Neg [[email protected]] has been kicked from #YourPants by Willie [Willie@P4A-A632E56F]: You should've picked the Blue wire.

Blue != blue?

@ghost ghost assigned embolalia Jan 9, 2012
embolalia added a commit that referenced this issue Jan 9, 2012
Other bots listed in the config file will now be ignored by .bomb,
as will the bot itself.
I'm not entirely sure where bug #8 is coming from, as the input and color
are both .lower()'ed. Tryed adding .rstrip(' '), in case that's the problem,
but it's still not quite right. Also still does not reject non-existant colors.
@tyrope
Copy link
Author

tyrope commented Jan 10, 2012

Why isn't this closed? Anyway,

Non-existant wires shouldn't trigger an explosion.

tyrope added a commit that referenced this issue Jan 13, 2012
Signed-off-by: Dimitri Molenaars <[email protected]>
@tyrope tyrope closed this as completed Jan 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Things to squish; generally used for issues
Projects
None yet
Development

No branches or pull requests

2 participants