Skip to content

minor fixes for pep8 in various files#42152

Merged
vbraun merged 2 commits into
sagemath:developfrom
fchapoton:minor_various_pep8
May 18, 2026
Merged

minor fixes for pep8 in various files#42152
vbraun merged 2 commits into
sagemath:developfrom
fchapoton:minor_various_pep8

Conversation

@fchapoton

Copy link
Copy Markdown
Contributor

about missing or spurious empty lines, etc

📝 Checklist

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

@github-actions

github-actions Bot commented May 10, 2026

Copy link
Copy Markdown

Documentation preview for this PR (built with commit d8d2a10; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

Comment thread src/sage/combinat/growth.py Outdated

@dcoudert dcoudert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The lint error seems unrelated

Comment thread src/sage/features/sagemath.py
Co-authored-by: Martin Rubey <axiomize@yahoo.de>
@cxzhong

cxzhong commented May 11, 2026

Copy link
Copy Markdown
Contributor

I think this is ok

@mantepse

Copy link
Copy Markdown
Contributor

@dcoudert still has a question unanswered.

@dcoudert dcoudert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

@vbraun vbraun merged commit c7b3f54 into sagemath:develop May 18, 2026
27 checks passed
@fchapoton fchapoton deleted the minor_various_pep8 branch May 19, 2026 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants