Skip to content

Matthew Eernisse mde

Organizations

Node Firm
mde commented on issue nlf/riakpbc#70
Matthew Eernisse

I was assuming too. Might be worth checking, but at least the plain-vanilla error case works correctly now. Thanks for the merge!

mde opened pull request nlf/riakpbc#71
Matthew Eernisse
Fixes issue #70, adds support for domains
1 commit with 4 additions and 0 deletions
mde pushed to master at mde/riakpbc
Matthew Eernisse
  • Matthew Eernisse 9d109f6
    Fixes issue #70, adds support for domains
mde opened issue nlf/riakpbc#70
Matthew Eernisse
Domains do not work
mde created tag v0.7.20 at jakejs/jake
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
  • Matthew Eernisse 20a3e8f
    Prefix ns for aliases in default ns
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
  • Matthew Eernisse 5acba1f
    Need ref to enclosing namespace
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
  • Matthew Eernisse 3aaf3e7
    Break up PublishTask, clean up namespace
mde commented on issue geddy/geddy#619
Matthew Eernisse

This would be fantastic. We have some work going on pluggable generators in this repo from @der-On: https://github.com/geddy/geddy-gen-gen But I wo…

mde commented on pull request jakejs/jake#225
Matthew Eernisse

Merged, thanks!

mde pushed to master at jakejs/jake
Matthew Eernisse
mde merged pull request jakejs/jake#225
Matthew Eernisse
Naming of WatchTask(s)
2 commits with 3 additions and 3 deletions
mde commented on pull request jakejs/jake#225
Matthew Eernisse

I have no idea how this got dropped. This is absolutely something people have asked for on other occasions. I'll get this fixed up and merged ASAP. :)

mde commented on issue geddy/geddy#618
Matthew Eernisse

The tests for our ORM layer (called Model) have a case that is exactly what you're attempting to do, but we use a Person model with a "Friendship" …

mde commented on issue jakejs/jake#262
Matthew Eernisse

Thanks so much for following up on this. Node generally does a good job of being cross-platform, but it still falls down pretty hard in some areas. #…

mde closed issue jakejs/jake#263
Matthew Eernisse
watchTask breaks without the presence of default exclusion directories
mde commented on issue jakejs/jake#263
Matthew Eernisse

Fixed in 9239f6c

mde pushed to master at jakejs/jake
Matthew Eernisse
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
mde created tag v0.7.19 at jakejs/jake
mde pushed to v0.7 at jakejs/jake
Matthew Eernisse
  • Matthew Eernisse 9239f6c
    WatchTask, only exclude dirs that exist
mde commented on pull request jakejs/jake#264
Matthew Eernisse

Definitely needs some pretty extensive tests. I don't have plans to implement it another way, so we can certainly merge this branch once there's re…

mde commented on issue geddy/geddy#617
Matthew Eernisse

That error is a bit misleading -- it's just a method that's mixed onto the object it's getting called on. And it looks like I misread the actual m…

mde commented on pull request jakejs/jake#264
Matthew Eernisse

Hm, rather than just bandaiding it by adding another Boolean flag, this really should be a three-state variable.

mde commented on pull request jakejs/jake#261
Matthew Eernisse

Actually now that you mention it, combining seems like the right idea. It seems a little too copy-pasta if it's repeated.

mde commented on pull request jakejs/jake#264
Matthew Eernisse

Nope, that's totally legit. So you're really dealing with only one level of prereqs. That's fine, but whatever we implement needs to take deeper hi…

mde commented on pull request jakejs/jake#264
Matthew Eernisse

@veldsla, are you using the Task API to add prereqs on the fly? Or are you just calling task and declaring new tasks?

Something went wrong with that request. Please try again.