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

Fixes #11981 #12009

Closed
wants to merge 4 commits into from
Closed

Fixes #11981 #12009

wants to merge 4 commits into from

Conversation

mohit3011
Copy link
Contributor

<If this pull request fixes an issue please indicate which issue by typing "Fixes #NNNN" below.>

@jksuom
Copy link
Member

jksuom commented Jan 2, 2017

There is another pending PR #12008 on this issue. That will probably settle the issue.

BTW, for your next PR it is recommended to create a new branch with a name different from 'master' since that may lead to confusion.

@mohit3011
Copy link
Contributor Author

mohit3011 commented Jan 2, 2017

Sorry I read your comment after the tests had been initiated . I am getting the same error as #12008 got , If there is no problem then can you look into what is wrong in my PR.

@jksuom
Copy link
Member

jksuom commented Jan 2, 2017

"File contains trailing whitespace: /home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/sympy-1.0.1.dev0-py2.7.egg/sympy/simplify/tests/test_powsimp.py, line 297"

This is from the Travis CI log https://travis-ci.org/sympy/sympy/jobs/188216580,

You can catch such errors before pushing to GitHub by running bin/test quality.

@mohit3011
Copy link
Contributor Author

Thank you very much for guiding me.

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