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

some pep8 fixes in combinat #35917

Merged

vbraun merged 3 commits into sagemath:develop from fchapoton:pep8_in_combinat_1

Jul 20, 2023

Contributor

fchapoton commented Jul 8, 2023

📚 Description

just fixing a few pycodestyle warnings in combinat folder

📝 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


          some pep8 fixes in combinat

646d9f5

fchapoton added c: combinatorics t: refactoring s: needs review labels

mkoeppe reviewed

View reviewed changes

src/sage/combinat/fully_packed_loop.py Outdated Show resolved Hide resolved

mkoeppe reviewed

View reviewed changes

src/sage/combinat/posets/incidence_algebras.py Outdated Show resolved Hide resolved

mkoeppe reviewed

View reviewed changes

src/sage/combinat/ribbon_shaped_tableau.py Outdated Show resolved Hide resolved

mkoeppe reviewed

View reviewed changes

src/sage/combinat/subsets_hereditary.py Show resolved Hide resolved

mkoeppe reviewed

View reviewed changes

src/sage/combinat/t_sequences.py Show resolved Hide resolved

fchapoton added 2 commits

July 9, 2023 19:58


          Merge branch 'develop' into pep8_in_combinat_1

85fed41


          suggested fixes done

94881f8

github-actions bot commented Jul 9, 2023

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

mkoeppe approved these changes

View reviewed changes

mkoeppe added s: positive review and removed s: needs review labels

vbraun merged commit 0c9ab0e into sagemath:develop

12 of 13 checks passed

mkoeppe added this to the sage-10.1 milestone

kwankyu removed the s: positive review label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment