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

PR: Add more tests to this package #50

Merged
merged 10 commits into from
May 5, 2017

Conversation

dalthviz
Copy link
Member

@dalthviz dalthviz commented Apr 2, 2017

Fixes #47

@dalthviz dalthviz added this to the v0.1 milestone Apr 2, 2017
@dalthviz dalthviz self-assigned this Apr 2, 2017
@dalthviz dalthviz requested a review from ccordoba12 April 2, 2017 00:38
@dalthviz dalthviz changed the title PR: Add more test for the notebook [WIP] PR: Add more test for the notebook Apr 2, 2017
@dalthviz dalthviz changed the title [WIP] PR: Add more test for the notebook PR: Add more test for the notebook Apr 11, 2017
@dalthviz
Copy link
Member Author

@ccordoba12 this is ready to merge?

@ccordoba12 ccordoba12 changed the title PR: Add more test for the notebook PR: Add more tests for the notebook May 5, 2017
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job here @dalthviz!!

@ccordoba12 ccordoba12 changed the title PR: Add more tests for the notebook PR: Add more tests May 5, 2017
@ccordoba12 ccordoba12 changed the title PR: Add more tests PR: Add more tests to this package May 5, 2017
@ccordoba12 ccordoba12 merged commit f352f90 into spyder-ide:master May 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants