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

Remove new assumptions #122

Merged
merged 3 commits into from Jan 18, 2016

Conversation

1 participant
@skirpichev
Copy link
Collaborator

skirpichev commented Aug 11, 2015

No description provided.

@skirpichev skirpichev force-pushed the remove-new-assumptions branch 2 times, most recently from 3fd5233 to df7c96d Aug 22, 2015

@skirpichev skirpichev force-pushed the remove-new-assumptions branch from df7c96d to 88d5a86 Oct 1, 2015

@skirpichev skirpichev force-pushed the remove-new-assumptions branch from 90ed47d to 84f3c6b Nov 24, 2015

@skirpichev skirpichev force-pushed the remove-new-assumptions branch 4 times, most recently from b9fda94 to 6a6e084 Dec 16, 2015

@skirpichev skirpichev force-pushed the remove-new-assumptions branch from 7f4f40a to 52ceb89 Jan 7, 2016

@skirpichev skirpichev force-pushed the remove-new-assumptions branch from 52ceb89 to e5202e1 Jan 13, 2016

@skirpichev

This comment has been minimized.

Copy link
Collaborator

skirpichev commented Jan 18, 2016

Perhaps, someday we can reintroduce it, but now the presence of new assumptions will just confuse users, see e.g. this.

skirpichev added a commit that referenced this pull request Jan 18, 2016

@skirpichev skirpichev merged commit 06c1c82 into master Jan 18, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.002%) to 91.89%
Details

@skirpichev skirpichev deleted the remove-new-assumptions branch Jan 18, 2016

@skirpichev skirpichev modified the milestone: 0.8.0 Sep 9, 2016

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