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

fix(Travis build): PhantomJS and SocketIO dependencies #1165

Merged
merged 1 commit into from
Jan 27, 2016

Conversation

trainerbill
Copy link
Contributor

changed PhantomJS and SocketIO dependencies to ~ to stop breaking changes.

@codydaig codydaig added this to the 0.5.0 milestone Jan 27, 2016
@codydaig codydaig self-assigned this Jan 27, 2016
@codydaig
Copy link
Member

LGTM

codydaig added a commit that referenced this pull request Jan 27, 2016
fix(Travis build): PhantomJS and SocketIO dependencies
@codydaig codydaig merged commit 271ec36 into meanjs:master Jan 27, 2016
@trainerbill trainerbill deleted the TravisFixes branch January 27, 2016 18:16
@ilanbiala
Copy link
Member

LGTM, not sure why those were not using ~ before.

@codydaig
Copy link
Member

@ilanbiala There seems to be a bunch that were/are like that. These in particular were breaking things so we fixed rapidly.

@trainerbill
Copy link
Contributor Author

Yeah, someone should probably do a dependency audit. I can't think of any reason why they wouldnt all be ~. Any breaking changes, ie minor/major upgrades, should be tested before.

@codydaig codydaig mentioned this pull request Jan 27, 2016
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.

3 participants