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

[ENH] Add Datasets widget #2557

Merged
merged 10 commits into from
Sep 21, 2017
Merged

[ENH] Add Datasets widget #2557

merged 10 commits into from
Sep 21, 2017

Conversation

lanzagar
Copy link
Contributor

@lanzagar lanzagar commented Sep 1, 2017

Description of changes

Widget moved.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov-io
Copy link

codecov-io commented Sep 1, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@8df4187). Click here to learn what that means.
The diff coverage is 50%.

@@            Coverage Diff            @@
##             master    #2557   +/-   ##
=========================================
  Coverage          ?   74.95%           
=========================================
  Files             ?      331           
  Lines             ?    58114           
  Branches          ?        0           
=========================================
  Hits              ?    43558           
  Misses            ?    14556           
  Partials          ?        0

@lanzagar lanzagar force-pushed the datasets branch 2 times, most recently from 6ae5189 to e560085 Compare September 5, 2017 13:51
@astaric
Copy link
Member

astaric commented Sep 19, 2017

@lanzagar, lint has a number of complaints. Can you take a look and fix them?

Some tests would be nice as well.

@astaric astaric changed the title [WIP] Move Data Sets widget from Prototypes [ENH] Add Datasets widget Sep 20, 2017
@astaric astaric merged commit 86cd792 into biolab:master Sep 21, 2017
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.

4 participants