barista version bump to 0.4.0 for PATCH, nested optional routes, new par...#612
barista version bump to 0.4.0 for PATCH, nested optional routes, new par...#612kieran wants to merge 1 commit into
Conversation
|
Looks like this breaks the build. Any idea off the top of your head what might not be backward compatible? |
|
looks like a templating issue - probably inherited from a previous commit? |
|
That's the application smoke test we do for each of the different templating engines. It generates a basic app for each one, then hits each of them with a few requests. I'm pretty sure this failure is due to the router change. The CI should run in all branches as well, so it's easy to dump large changes into feature branches and watch the CI. I should mention we've been having issues with Node 0.8 in the tests. I don't think the NPM backend service works at all with 0.8-circa clients. If your tests work in 0.10, you're good to go. |
|
hmm, seems to be an issue with the router helper generation in |
...ser
...sorry for the push to the wrong remote repo :-(