Skip to content

sage.graphs: More # optional and other modularization fixes#35647

Merged
vbraun merged 1 commit intosagemath:developfrom
mkoeppe:sage_graphs_modularization
May 28, 2023
Merged

sage.graphs: More # optional and other modularization fixes#35647
vbraun merged 1 commit intosagemath:developfrom
mkoeppe:sage_graphs_modularization

Conversation

@mkoeppe
Copy link
Copy Markdown
Contributor

@mkoeppe mkoeppe commented May 17, 2023

📚 Description

Part of:

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@github-actions
Copy link
Copy Markdown

github-actions bot commented May 18, 2023

Documentation preview for this PR is ready! 🎉
Built with commit: 979420d

dcoudert
dcoudert approved these changes May 18, 2023
View reviewed changes
Copy link
Copy Markdown
Collaborator

@dcoudert dcoudert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@mkoeppe
Copy link
Copy Markdown
Contributor Author

mkoeppe commented May 18, 2023

Thank you!

@vbraun vbraun merged commit 21e8f9f into sagemath:develop May 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dcoudert dcoudert dcoudert approved these changes

Assignees

@mkoeppe mkoeppe

Labels

c: graph theory t: refactoring

Projects

None yet

Milestone

sage-10.1

Development

Successfully merging this pull request may close these issues.

3 participants

@mkoeppe @dcoudert @vbraun