Opened 10 years ago

Closed 10 years ago

#12067 closed enhancement (fixed)

delete databases/bz2Pickle.py

Reported by: ohanar Owned by: mvngu
Priority: minor Milestone: sage-4.8
Component: doctest coverage Keywords: sd35
Cc: Merged in: sage-4.8.alpha6
Authors: R. Andrew Ohana Reviewers: Julian Rueth
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by ohanar)

Delete databases/bz2Pickle.py since it is not used anywhere in sage, provides no useful functionality, and hasn't been touched since the creation of the mercurial repo.

Part of metaticket #12024.

Attachments (1)

trac_12067.patch (1.6 KB) - added by ohanar 10 years ago.
fixed header

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by ohanar

  • Status changed from new to needs_review

comment:2 follow-up: Changed 10 years ago by saraedum

  • Status changed from needs_review to needs_info

Are these deleted because they're not used/useful anymore? (adding a description would make it possible to review this)

comment:3 in reply to: ↑ 2 Changed 10 years ago by ohanar

  • Description modified (diff)
  • Status changed from needs_info to needs_review

Replying to saraedum:

Are these deleted because they're not used/useful anymore?

Yup

comment:4 follow-up: Changed 10 years ago by saraedum

  • Reviewers set to Julian Rueth
  • Status changed from needs_review to needs_work

Strangely your patch doesn't have an "author" field. I assume the release manager needs that for the repository history. Could you try to re-export the patch with a full header?

Changed 10 years ago by ohanar

fixed header

comment:5 in reply to: ↑ 4 Changed 10 years ago by ohanar

  • Description modified (diff)
  • Status changed from needs_work to needs_review

Replying to saraedum:

Strangely your patch doesn't have an "author" field. I assume the release manager needs that for the repository history. Could you try to re-export the patch with a full header?

Done.

comment:6 Changed 10 years ago by saraedum

  • Status changed from needs_review to positive_review

comment:7 Changed 10 years ago by saraedum

  • Keywords sd35 added

comment:8 Changed 10 years ago by jdemeyer

  • Merged in set to sage-4.8.alpha6
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.