-
Notifications
You must be signed in to change notification settings - Fork 317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix build errors #442
Fix build errors #442
Conversation
Sigh, there is still an issue with the JupyterHub tests that I can't figure out. @minrk, do you have any ideas? The error looks like this:
In other words, it's giving that
This spawner works fine locally, and used to work fine on Travis. So I'm kind of stumped on what the issue is. Do you have any insights? |
Hmmm, I wonder if this could be due to a change on Travis. In the logs it says it doesn't allow use of |
That's possible, I'll see if I can figure our what's up in the morning. |
@minrk Thanks! I tried removing |
Yaay, it passes! |
Glad you found it. |
This should fix the build errors that are occuring on master currently... the issues were:
os.setpgrp()
from the fake spawner, because it doesn't work on Travis.This also splits the javascript tests into two separate groups, one for the formgrader and one for the nbextensions, which should hopefully make restarting tests a little less cumbersome.