Skip to content

fixing a few badly written loops #41149

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
Merged
merged 1 commit into from
Nov 11, 2025
Merged

fixing a few badly written loops #41149

merged 1 commit into from
Nov 11, 2025
+17 −11

Conversation

@fchapoton
Copy link
Contributor

@fchapoton fchapoton commented Nov 8, 2025

as these are creating a list inside just to loop over

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.

gmou3
gmou3 suggested changes Nov 8, 2025
View reviewed changes
@cxzhong
Copy link
Contributor

cxzhong commented Nov 10, 2025 •
edited
Loading

I think it is OK. It just preserves the original writting.

@fchapoton
Copy link
Contributor Author

fchapoton commented Nov 10, 2025

Thanks for the review

@vbraun vbraun merged commit c92c540 into sagemath:develop Nov 11, 2025
29 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gmou3 gmou3 gmou3 requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

t: refactoring

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fchapoton @cxzhong @gmou3 @vbraun