Opened 7 years ago
Closed 7 years ago
#20713 closed enhancement (fixed)
use six to import builtins (step 2/2)
Reported by: | chapoton | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-7.3 |
Component: | python3 | Keywords: | |
Cc: | tscrim, jdemeyer, jmantysalo, embray | Merged in: | |
Authors: | Frédéric Chapoton | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | a63248e (Commits, GitHub, GitLab) | Commit: | a63248edcc6653b828c858fa723f24847f16c720 |
Dependencies: | Stopgaps: |
Description
another bunch of py3 compatible imports of builtins
Change History (4)
comment:1 Changed 7 years ago by
Branch: | → public/20713 |
---|---|
Cc: | tscrim jdemeyer jmantysalo added |
Commit: | → a63248edcc6653b828c858fa723f24847f16c720 |
Status: | new → needs_review |
comment:3 Changed 7 years ago by
Reviewers: | → Travis Scrimshaw |
---|---|
Status: | needs_review → positive_review |
I wonder how much these imports of __builtin__
are really needed (i.e., we can't get rid of by just renaming some variables). Anyways, this works for now.
comment:4 Changed 7 years ago by
Branch: | public/20713 → a63248edcc6653b828c858fa723f24847f16c720 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
New commits:
use from six.moves.builtins import
use from six.moves.builtins, details of imports