Skip to content

Node.js all the things #9

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Node.js all the things #9

wants to merge 40 commits into from

Conversation

@fourkbomb
Copy link
Contributor

Well all the really weird bugs are fixed.

auscompgeek
auscompgeek reviewed Oct 18, 2014
View reviewed changes
aucgbot.js
Copy link
Owner

Choose a reason for hiding this comment

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

WHAT IF !nick, HUH? WHAT HAPPENS THEN!?

fourkbomb and others added 17 commits October 28, 2014 20:04
@fourkbomb
Fix flood control
118f2ae
@fourkbomb
Fix command things being buggy.
f7cc5d1
@mcpower
todo.jsm: save todos after delete
37a7774
@mcpower
todo.jsm: display todo indices when getting todos
9e8b274
@mcpower
todo.jsm: allow deletion of multiple todos
99e35bd
@mcpower
todo.jsm: allow deletion of todos by name
a34c051
@mcpower
updated tododel help string
90cbdae
@mcpower
todo.jsm: implemented todoins
ba0b124
@fourkbomb
Begin the port to Node JS. Bot works, around half the modules work.
b7dab05
@fourkbomb
Finish porting the second half of the modules.
994e0f2 
Some things may be incomplete. The following modules are not ported:
> bom.jsm
> elf.jsm
> halfop-bots.jsm
> infobot.jsm
@fourkbomb
Remove comments, update README.md
f7f1758
@fourkbomb
Something about "use strict" and crashes
ac664c2
@fourkbomb
Fix some weirdness when a single data event contains multiple lines
9053b61
@fourkbomb
Add run.sh because CLI switches are too complicated.
60eb0cb
@fourkbomb
Fix flood control
318dc8f
@fourkbomb
Fix command things being buggy.
0e460aa
@fourkbomb
Merge branch 'mcpower-master' into nodejs
ed438a1 
Conflicts:
	todo.jsm
@gnustomp
Copy link

Merge branch 'mcpower-master' into nodejs …

@keepcalm444

@auscompgeek
Copy link
Owner

Note to self: modules that are to be rewritten for node.js:

  • bom
  • elf
  • infobot
  • karma
  • wa (which isn't even in the repo...)

Someone give me a case-insensitive dict pls.

Also, those that require (no pun intended) slight modifications to work on node.js:

  • calcbot
  • date

@gnustomp
Copy link

gnustomp commented Jul 8, 2015

Wow.

@auscompgeek
Copy link
Owner

We can’t automatically merge this pull request.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement in progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fourkbomb @gnustomp @auscompgeek @mcpower