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 deprecated stuff from graphs #29518

Closed
jfraymond opened this issue Apr 16, 2020 · 7 comments
Closed

remove deprecated stuff from graphs #29518

jfraymond opened this issue Apr 16, 2020 · 7 comments

Comments

@jfraymond
Copy link

Goal: remove all the code of the graph module that has a deprecation warning referring to a ticket closed for at least one year.
These tickets are:

Note that the current ticket completely deletes the file graphs/linearextensions.py.

Component: graph theory

Keywords: deprecation

Author: Jean-Florent Raymond

Branch/Commit: b1f507f

Reviewer: Frédéric Chapoton

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

@jfraymond jfraymond added this to the sage-9.2 milestone Apr 16, 2020
@jfraymond
Copy link
Author

Author: Jean-Florent Raymond

@jfraymond
Copy link
Author

comment:1

On my machine the doctests for the graphs folder pass and the doc builds.

@fchapoton
Copy link
Contributor

comment:2

Looks good.

Could you please list in the ticket description field here the trac numbers of the tickets that introduced these deprecations ?

@jfraymond

This comment has been minimized.

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:4

ok, merci !

@vbraun
Copy link
Member

vbraun commented Apr 19, 2020

Changed branch from u/gh-jfraymond/remove-deprecated-stuff-in-graphs to b1f507f

@vbraun vbraun closed this as completed in 9980a1f Apr 19, 2020
@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.1 May 2, 2020
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

4 participants