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

Adding how to run tests on developer notes. #42

Merged
merged 14 commits into from
May 11, 2017
Merged

Conversation

raphaeltimbo
Copy link
Collaborator

This adds an small paragraph for details on how to run the doctests.

@josephcslater
Copy link
Member

Raphael- do you know what's going on with the failed checks? My merge from Master were typos, not code, that I didn't want clashes with. The travis log says it's the doc string in euler beam... I didn't know anyone was touching that.

@raphaeltimbo
Copy link
Collaborator Author

@josephcslater I was taking a look at this, and I think that the revert from #41 did not happen, since it was not merged (if we look at the master branch the code from #39 is there).
I saw the errors and they were due to a print statement and 2 figures that the output was not present at the docstrings.
I think we can merge this pull request and close #41 , this way we can use the code that has already been written by @AnushaAnisetti on #39 .

@josephcslater josephcslater merged commit 1c713ea into master May 11, 2017
@josephcslater josephcslater deleted the developer_notes branch May 11, 2017 16:40
@josephcslater
Copy link
Member

Thank you for figuring this out. I expected to spend time this weekend on it.

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.

None yet

2 participants