Moved messages to config.yml #3

Closed
johnoliverdriscoll wants to merge 1 commit from master into master
johnoliverdriscoll commented 2012-07-07 03:49:43 +00:00 (Migrated from github.com)
No description provided.
LadyCailin commented 2013-11-02 03:21:31 +00:00 (Migrated from github.com)

Anybody gonna fix this, or am I gonna have to? lol

Anybody gonna fix this, or am I gonna have to? lol
totemo commented 2013-11-05 08:01:37 +00:00 (Migrated from github.com)

Yep. Go for it if you feel motivated. I got distracted.

Yep. Go for it if you feel motivated. I got distracted.
LadyCailin commented 2013-11-05 13:16:29 +00:00 (Migrated from github.com)

I started to merge it, but it's non-trivial.

On Nov 5, 2013, at 2:01, totemo [email protected] wrote:

Yep. Go for it if you feel motivated. I got distracted.


Reply to this email directly or view it on GitHub.

I started to merge it, but it's non-trivial. On Nov 5, 2013, at 2:01, totemo [email protected] wrote: > Yep. Go for it if you feel motivated. I got distracted. > > — > Reply to this email directly or view it on GitHub.
Deaygo commented 2013-11-05 14:25:56 +00:00 (Migrated from github.com)

This is probably easiest to just re-do since there were a fair number of changes made since this.

This is probably easiest to just re-do since there were a fair number of changes made since this.
johnoliverdriscoll commented 2013-11-05 16:34:41 +00:00 (Migrated from github.com)

Just reject the request. Also, thanks go to deaygo for dropping the ball on this. He asked me to make this change then never looked at it or merged it. Good luck guys, you need it.

Just reject the request. Also, thanks go to deaygo for dropping the ball on this. He asked me to make this change then never looked at it or merged it. Good luck guys, you need it.
LadyCailin commented 2013-11-06 16:09:59 +00:00 (Migrated from github.com)

Given that this is so hard to merge, I'm going to go ahead and close it. Sorry we didn't get to this sooner, we really did drop the ball.

Given that this is so hard to merge, I'm going to go ahead and close it. Sorry we didn't get to this sooner, we really did drop the ball.
johnoliverdriscoll commented 2013-11-07 03:02:37 +00:00 (Migrated from github.com)

No apologies necessary to me, I don't even play minecraft anymore. But your players deserve better work out of you (the collective you). You are the shoot-yourself-in-the-foot kind of lazy. At least go for making-work-easier-on-yourself kind of lazy.

No apologies necessary to me, I don't even play minecraft anymore. But your players deserve better work out of you (the collective you). You are the shoot-yourself-in-the-foot kind of lazy. At least go for making-work-easier-on-yourself kind of lazy.
Deaygo commented 2013-11-07 03:04:18 +00:00 (Migrated from github.com)

I am currently re-implementing your config.yml and will hopefully be pushing it tonight. :)

I am currently re-implementing your config.yml and will hopefully be pushing it tonight. :)
johnoliverdriscoll commented 2013-11-07 03:06:34 +00:00 (Migrated from github.com)

Sounds familiar

Sounds familiar
Deaygo commented 2013-11-07 04:08:04 +00:00 (Migrated from github.com)

github.com/NerdNu/ModReq@a604315a9a still some testing to be done, but should be mostly good to go.

https://github.com/NerdNu/ModReq/commit/a604315a9a776502d18f0a198e4e179ea6b4e678 still some testing to be done, but should be mostly good to go.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
NerdNu/ModReq!3
No description provided.