search

nomicness / a-whole-new-world

2 stars 7 forks source link

New bot features #92

Closed jimmyhmiller closed 8 years ago

jimmyhmiller commented 8 years ago

The diff is going to look crazy because I moved things to src.

I'm not sure how clear the idea behind the code will be. But if you checkout the allocate function, it is pretty straight forward. The goal is to be able to write all commands in as straight forward a way as possible.

After this I'm going to have some bulk edits just modernizing things.

Feel free to ask any questions.

ArchmageInc commented 8 years ago

Wow, okay, so only one question: Does this break anything @jimmyhmiller?

dilbertKocik commented 8 years ago

lol, yeah I was wondering the same thing. It looks pretty solid. But it's a lot of code to read through so who knows.

jimmyhmiller commented 8 years ago

I didn't test everything exhaustively, but I think it should work. If things do break, I'm happy to fix them. I've got a lot more planned for this, hopefully we can start having new additions become easier and easier.

jimmyhmiller commented 8 years ago

@ArchmageInc @dilbertKocik Any objections to me merging it?

jimmyhmiller commented 8 years ago

I actually did find a minor bug but have fixed it.

ArchmageInc commented 8 years ago

This is now built and pushed as archmageinc/nomic-bot:latest

ArchmageInc commented 8 years ago

archmageinc/nomic-bot:latest has been deployed. The previous version was archmageinc/nomic-bot:sawmill.