Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Update dependencies #950

Closed
wants to merge 1 commit into from
Closed

Update dependencies #950

wants to merge 1 commit into from

Conversation

ilanbiala
Copy link
Member

This is a WIP PR.

@ilanbiala
Copy link
Member Author

@simison I don't really see any major benefits over forever.

@codydaig
Copy link
Member

@simison Unitech doesn't credit their contributors anymore, so I prefer not to support a project that doesn't give credit to the authors.

@lirantal
Copy link
Member

lirantal commented Oct 1, 2015

we're using PM2 at work as well.

@ilanbiala
Copy link
Member Author

@lirantal Node 0.12 only has failing tests. However, I'd prefer to wait for us to up the test coverage before finalizing this because then we'll have to do less testing before merging.

@lirantal
Copy link
Member

lirantal commented Oct 2, 2015

Cool, fine by me.

@ilanbiala
Copy link
Member Author

Not a great idea to do this at once, so I'm closing and we'll do it incrementally as much as we can.

@ilanbiala ilanbiala deleted the dependency-update branch October 27, 2015 19:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants