Correctly deprecate limit_to_all #10198

Merged
merged 1 commit into from Jan 26, 2017

Projects

None yet

2 participants

@Carreau
Member
Carreau commented Jan 24, 2017

At least emit a warning.

Should likely be backported to 5.x.

@Carreau Carreau added this to the 5.2 milestone Jan 24, 2017
@Carreau Carreau Correctly deprecate limit_to_all
8970389
@Carreau
Member
Carreau commented Jan 26, 2017

Ok I don't like to do this, but i'm going to self merge if there is no reviews.

@takluyver takluyver merged commit 9d720e3 into ipython:master Jan 26, 2017

4 checks passed

codecov/patch 50.00% of diff hit (target 0.00%)
Details
codecov/project 66.22% (-0.01%) compared to 28d41ca
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@takluyver
Member

@meeseeksdev backport to 5.x

@meeseeksdev meeseeksdev pushed a commit that referenced this pull request Jan 26, 2017
@takluyver takluyver + MeeseeksDev[bot] Backport PR #10198: Correctly deprecate limit_to_all
At least emit a warning.

Should likely be backported to 5.x.
dfc832f
@Carreau Carreau deleted the Carreau:limit-all-deprecation branch Jan 26, 2017
@Carreau Carreau added the backported label Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment