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

Update dependencies, bring .travis config into 2015 #96

Merged
merged 2 commits into from
Apr 29, 2015

Conversation

cliftonc
Copy link
Contributor

Unclear if this project has been abandoned (be happy to help out), but all of the dependencies and core configuration are a very long way out of date.

This request updates the three core dependencies, and updates Travis to work on current versions of Node that most people are likely to be running in production.

@andreas-marschke
Copy link

+1

@rfk
Copy link
Contributor

rfk commented Apr 28, 2015

Thanks! It's certainly not abandoned, we're using it in production, although I suspect it's mostly in "things just work so it doesn't get much attention" mode these days.

@zaach are you the closest we have to a "module owner" on this project today?

@cliftonc
Copy link
Contributor Author

The main reason (selfishly) I want this to be accepted, is that the old version of the plist dependency contains a 26mb test file that is making a lot of our services much bigger than they need to be, and with a very distributed team on sometimes not great connections this does surprisingly make a difference.

We're starting to build a lot around this and so was also keen to give it a nudge to see if there are signs of life.

@@ -1,5 +1,6 @@
language: node_js

node_js:
- 0.6
- 0.8
- "1.6"
Copy link
Contributor

Choose a reason for hiding this comment

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

I can't find it in the travis docs, what does "1.6" build here?

Copy link
Contributor

Choose a reason for hiding this comment

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

@rfk: I reckon that'd be an io.js version number.

@rfk
Copy link
Contributor

rfk commented Apr 29, 2015

Otherwise this LGTM and I don't see much risk to merging

@rfk
Copy link
Contributor

rfk commented Apr 29, 2015

anyway, travis says it's green so imma merge this thing; thanks again @cliftonc

rfk added a commit that referenced this pull request Apr 29, 2015
Update dependencies, bring .travis config into 2015
@rfk rfk merged commit aa71416 into mozilla:master Apr 29, 2015
@cliftonc
Copy link
Contributor Author

Fantastic - thanks!

@zaach
Copy link
Contributor

zaach commented Apr 29, 2015

New version published. 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants