Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Express update & fixing tests #81

wants to merge 4 commits into


None yet
2 participants

I wanted to hack on tobi, but the dev dependencies are pulling in a now-unpublished version of express. So I bumped that, and a bunch of tests are failing (IDK if they were failing on older versions as well).

I've fixed some of the tests, and I'm gonna hopefully go through the rest and fix them all, but feel free to merge at any time :)

To be honest, I don't completely know what I'm doing with all the code, so if you catch me doing something crazy, let me know.

iangreenleaf added some commits May 31, 2012

Remove .include
It looks like should.js has changed support for .include(), so this
assertion won't work correctly right now.

This helped me run on an app running express 2.5. it would not start correctly without this patch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment