Skip to content
This repository has been archived by the owner on Nov 27, 2019. It is now read-only.

Express four support #3

Closed
wants to merge 4 commits into from

Conversation

nickdesaulniers
Copy link

review? @kumar303

Figured this was worthy of a version major bump

"superagent": "^0.20.0",
"underscore": "^1.7.0",
"jwt-simple": "^0.1.0",
"body-parser": "^1.9.0"
Copy link
Author

Choose a reason for hiding this comment

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

I should get rid of the ^. I don't know what they do other than cause errors.

@chuckharmston
Copy link

@kumar303 Is out for a little while, FWIW. Maybe @andymckay or @muffinresearch?

@nickdesaulniers
Copy link
Author

Not high priority. I can use my fork for the time being.

@kumar303
Copy link
Contributor

kumar303 commented Apr 1, 2015

Oops, this almost fell through the cracks. Thanks for the patch!

It didn't merge cleanly but I fixed the conflicts and got it in:

15cf388
386943d
eda7d6c

@kumar303 kumar303 closed this Apr 1, 2015
@kumar303
Copy link
Contributor

kumar303 commented Apr 1, 2015

Nick, you should update your fork ASAP to get some security fixes

@kumar303 kumar303 mentioned this pull request Apr 1, 2015
@nickdesaulniers nickdesaulniers deleted the expressfour branch April 2, 2015 16:16
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.

3 participants