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

Add responses to some new phrases #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Tocacar
Copy link

@Tocacar Tocacar commented Jul 28, 2012

Starting to learn a bit of node.js. Here are a few responses to some phrases used from time to time in the channel. I have no way to test whether they work, but they should be OK I have just recycled pre-existing code on the page.

Kim Rowan added 2 commits July 28, 2012 18:10
Unfortunately, the regex is picking up 'ls' anywhere within the string, rather than just "ls" on its own, which is what I had intended.  Learned something new about regexes!  For now, its quicker to remove it as its driving people nuts.
@Tocacar
Copy link
Author

Tocacar commented Aug 2, 2012

Have updated PR to remove detection of 'ls'. I got the regex wrong and its driving everyone mad. Sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant