Skip to content

src/sage/combinat/root_system/coxeter_group.py: Fix typo in lazy_import#36698

Merged
vbraun merged 2 commits intosagemath:developfrom
mkoeppe:coxeter_group_lazy_import_typo
Nov 12, 2023
Merged

src/sage/combinat/root_system/coxeter_group.py: Fix typo in lazy_import#36698
vbraun merged 2 commits intosagemath:developfrom
mkoeppe:coxeter_group_lazy_import_typo

Conversation

@mkoeppe
Copy link
Copy Markdown
Contributor

@mkoeppe mkoeppe commented Nov 11, 2023

📝 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 Nov 11, 2023

Documentation preview for this PR (built with commit ddc6a46; changes) is ready! 🎉

dcoudert
dcoudert approved these changes Nov 11, 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 Nov 11, 2023

Thanks!

@fchapoton
Copy link
Copy Markdown
Contributor

fchapoton commented Nov 12, 2023

this is no longer clean, after Volker rebasing develop

@vbraun vbraun merged commit 883e05f into sagemath:develop Nov 12, 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: combinatorics p: blocker / 1

Projects

None yet

Milestone

sage-10.2

Development

Successfully merging this pull request may close these issues.

4 participants

@mkoeppe @fchapoton @dcoudert @vbraun