Opened 5 years ago

Closed 5 years ago

#21150 closed enhancement (fixed)

py3 imports using six for StringIO and cStringIO

Reported by: chapoton Owned by:
Priority: major Milestone: sage-7.3
Component: python3 Keywords:
Cc: tscrim, embray, jdemeyer, aapitzsch Merged in:
Authors: Frédéric Chapoton Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 367f931 (Commits, GitHub, GitLab) Commit: 367f9313640cad2b4e1d57cfe60be8f29c05c651
Dependencies: Stopgaps:

Status badges

Description

as another step towards python3

part of #16069

Change History (8)

comment:1 Changed 5 years ago by chapoton

  • Branch set to public/21150
  • Commit set to 742bf2d4fd2ead81eb904cdd81903750bc03637e

New commits:

742bf2dpy3 imports for StringIO and cStringIO

comment:2 Changed 5 years ago by git

  • Commit changed from 742bf2d4fd2ead81eb904cdd81903750bc03637e to ac54b1a0fcb90174f8aafed7fb45970b9b37acf2

Branch pushed to git repo; I updated commit sha1. New commits:

ac54b1atrac 21150 adding absolute_import in explain_pickle

comment:3 Changed 5 years ago by chapoton

  • Cc tscrim embray jdemeyer aapitzsch added
  • Status changed from new to needs_review

bot is green, please review

comment:4 Changed 5 years ago by tscrim

  • Reviewers set to Travis Scrimshaw

If you are going to change if len(L) > 0:, then you might as well change it to the faster if L:. Otherwise LGTM.

comment:5 Changed 5 years ago by git

  • Commit changed from ac54b1a0fcb90174f8aafed7fb45970b9b37acf2 to 367f9313640cad2b4e1d57cfe60be8f29c05c651

Branch pushed to git repo; I updated commit sha1. New commits:

367f931trac #21150 detail

comment:6 Changed 5 years ago by chapoton

done

comment:7 Changed 5 years ago by tscrim

  • Status changed from needs_review to positive_review

Thanks.

comment:8 Changed 5 years ago by vbraun

  • Branch changed from public/21150 to 367f9313640cad2b4e1d57cfe60be8f29c05c651
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.