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

Remove a lot of deprecated code #24103

Closed
jdemeyer opened this issue Oct 25, 2017 · 21 comments
Closed

Remove a lot of deprecated code #24103

jdemeyer opened this issue Oct 25, 2017 · 21 comments

Comments

@jdemeyer
Copy link

Remove code deprecated in #2536, #13605, #14469, #16930, #16983, #16878, #17229, #17443, #17451, #17686, #18558, #19694, #19888, #20015, #20253, #20318, #20442, #21141, #21145, #21148, #21376, #21435

This includes a lot of deprecations for cmp instead of key for sorting.

Component: misc

Author: Jeroen Demeyer

Branch/Commit: aae6e54

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/24103

@jdemeyer jdemeyer added this to the sage-8.1 milestone Oct 25, 2017
@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

@jdemeyer
Copy link
Author

New commits:

1b28d3dRemove a lot of deprecated code

@jdemeyer
Copy link
Author

Commit: 1b28d3d

@jdemeyer
Copy link
Author

Author: Jeroen Demeyer

@jdemeyer

This comment has been minimized.

@tscrim
Copy link
Collaborator

tscrim commented Oct 25, 2017

comment:5

Lots of failing tests because you missed this line in words.py:

self.cmp_letters = self._cmp_letters  # deprecated

Actually, you should be able to remove cmp_letters as well.

@tscrim
Copy link
Collaborator

tscrim commented Oct 25, 2017

Reviewer: Travis Scrimshaw

@jdemeyer
Copy link
Author

comment:6

Adding #21376

@jdemeyer

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 26, 2017

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

e26c8e8Remove a lot of deprecated code

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 26, 2017

Changed commit from 1b28d3d to e26c8e8

@tscrim
Copy link
Collaborator

tscrim commented Oct 26, 2017

comment:10

I would say either the failing doctest in doctest/control.py is invalid because it was checking the deprecation warning or it is now the correct output. However, since you wrote that, I guess you would know. :P

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 27, 2017

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

a3589deRemove a lot of deprecated code

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 27, 2017

Changed commit from e26c8e8 to a3589de

@tscrim
Copy link
Collaborator

tscrim commented Oct 28, 2017

comment:13

LGTM.

@vbraun
Copy link
Member

vbraun commented Oct 29, 2017

comment:14

Merge conflict

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 30, 2017

Changed commit from a3589de to aae6e54

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Oct 30, 2017

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

aae6e54Remove a lot of deprecated code

@jdemeyer
Copy link
Author

comment:17

I reverted the conflicting change in words.py (involving cmp_letters). This way, the rest of this branch can still be merged.

@vbraun
Copy link
Member

vbraun commented Nov 1, 2017

Changed branch from u/jdemeyer/remove_a_lot_of_deprecated_code to aae6e54

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

No branches or pull requests

3 participants