Skip to content
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

Pinning against specific version of Jupyterhub to fix red. #396

Merged
merged 1 commit into from
Apr 10, 2017

Conversation

stefanoborini
Copy link
Contributor

@stefanoborini stefanoborini commented Apr 10, 2017

Apparently something has changed upstream in jupyterhub so that it makes our master red.
I pinned against 2d1a45f0190059ef436c2f97dc8d6e391eb2d139 for now, but the problem needs to be investigated further.

@codecov-io
Copy link

codecov-io commented Apr 10, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@5dadfdf). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #396   +/-   ##
=========================================
  Coverage          ?   95.17%           
=========================================
  Files             ?       94           
  Lines             ?     3996           
  Branches          ?      256           
=========================================
  Hits              ?     3803           
  Misses            ?      139           
  Partials          ?       54

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5dadfdf...60008f6. Read the comment docs.

@stefanoborini
Copy link
Contributor Author

Error due to missing coverage. Good to go.

@stefanoborini stefanoborini merged commit 5725ffe into master Apr 10, 2017
@stefanoborini stefanoborini deleted the test-branch branch April 10, 2017 10:18
@stefanoborini
Copy link
Contributor Author

Further kept track on pr #396

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants