Opened 8 years ago
Closed 8 years ago
#14216 closed enhancement (fixed)
clean-up of parts of combinat
Reported by: | chapoton | Owned by: | sage-combinat |
---|---|---|---|
Priority: | minor | Milestone: | sage-5.8 |
Component: | combinatorics | Keywords: | clean-up |
Cc: | sage-combinat | Merged in: | sage-5.8.beta3 |
Authors: | Frédéric Chapoton | Reviewers: | Nathann Cohen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Here is a small patch that
- removes duplicate or unused imports
- changes the raise syntax to Python 3 syntax
- removes unused variables
in a few combinat files
Attachments (1)
Change History (5)
Changed 8 years ago by
comment:1 Changed 8 years ago by
- Status changed from new to needs_review
comment:2 Changed 8 years ago by
- Status changed from needs_review to positive_review
comment:3 Changed 8 years ago by
- Reviewers set to Nathann Cohen
comment:4 Changed 8 years ago by
- Merged in set to sage-5.8.beta3
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Makes sense, passes tests, good to go ! The only problem with this ticket is that it should be named "clean up parts of combinat 1/3457", because there is that much left to do
:-P
Cc me if you write any other such ticket !
Nathann