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.

Sign up for GitHub

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

Jump to bottom

pycodestyle cleanup in src/sage/graphs/graph.py (part 3) #34317

Closed
dcoudert opened this issue Aug 9, 2022 · 8 comments
Closed

pycodestyle cleanup in src/sage/graphs/graph.py (part 3) #34317

dcoudert opened this issue Aug 9, 2022 · 8 comments
Labels
c: graph theory p: minor / 4 t: enhancement
Milestone
sage-9.7

Comments

@dcoudert
Copy link
Contributor

dcoudert commented Aug 9, 2022

Continue the cleaning of src/sage/graphs/graph.py after #34315 and #34316.

Depends on #34315
Depends on #34316

Component: graph theory

Author: David Coudert

Branch/Commit: 258eb97

Reviewer: Sebastian Oehms

Issue created by migration from https://trac.sagemath.org/ticket/34317
@dcoudert dcoudert added this to the sage-9.7 milestone Aug 9, 2022
@dcoudert
Copy link
Contributor Author

dcoudert commented Aug 9, 2022

Commit: 258eb97

@dcoudert
Copy link
Contributor Author

dcoudert commented Aug 9, 2022

New commits:

93d51cetrac #34315: clean src/sage/graphs/graph.py - part 1
fed43eatrac #34316: clean src/sage/graphs/graph.py - part 2
258eb97trac #34317: clean src/sage/graphs/graph.py - part 3

@dcoudert
Copy link
Contributor Author

dcoudert commented Aug 9, 2022

Branch: public/graphs/34317_graph3

@dcoudert
Copy link
Contributor Author

dcoudert commented Aug 9, 2022

comment:2

It remains very few in graph.py.

@soehms
Copy link
Member

soehms commented Aug 16, 2022

comment:3

LGTM

@soehms
Copy link
Member

soehms commented Aug 16, 2022

Reviewer: Sebastian Oehms

@dcoudert
Copy link
Contributor Author

comment:4

Thank you.

@vbraun
Copy link
Member

vbraun commented Aug 30, 2022

Changed branch from public/graphs/34317_graph3 to 258eb97

@vbraun vbraun closed this as completed in 72cd77c Aug 30, 2022
This was referenced Aug 29, 2022
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: graph theory p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-9.7
Development

No branches or pull requests
4 participants
@vbraun @mkoeppe @dcoudert @soehms