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

A Better Update to Issue #1089 #1100

Merged
merged 1 commit into from Oct 26, 2016

Conversation

Projects
None yet
3 participants
@ShikharJ
Member

ShikharJ commented Oct 25, 2016

My local system gave me a lot of issues (clock skew, installation etc.). I believe most of them are resolved now and hence i would like to put forward a fresh pull request for #1089.

Fixes #1089

@ShikharJ

This comment has been minimized.

Show comment
Hide comment
@ShikharJ

ShikharJ Oct 26, 2016

Member

@isuruf @certik The checks have finally passed. Please suggest me any further changes ( if necessary ).

Member

ShikharJ commented Oct 26, 2016

@isuruf @certik The checks have finally passed. Please suggest me any further changes ( if necessary ).

@isuruf isuruf merged commit d178e40 into symengine:master Oct 26, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ShikharJ ShikharJ deleted the ShikharJ:Issue1089 branch Oct 26, 2016

@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Oct 26, 2016

Contributor

Looks good, thanks @ShikharJ.

Contributor

certik commented Oct 26, 2016

Looks good, thanks @ShikharJ.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment