Skip to content
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.

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

delete databases/bz2Pickle.py #12067

Closed
ohanar opened this issue Nov 21, 2011 · 10 comments
Closed

delete databases/bz2Pickle.py #12067

ohanar opened this issue Nov 21, 2011 · 10 comments

Comments

@ohanar
Copy link
Member

ohanar commented Nov 21, 2011

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.

Component: doctest coverage

Keywords: sd35

Author: R. Andrew Ohana

Reviewer: Julian Rueth

Merged: sage-4.8.alpha6

Issue created by migration from https://trac.sagemath.org/ticket/12067

@saraedum
Copy link
Member

comment:2

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

@ohanar
Copy link
Member Author

ohanar commented Dec 19, 2011

comment:3

Replying to @saraedum:

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

Yup

@ohanar

This comment has been minimized.

@saraedum
Copy link
Member

Reviewer: Julian Rueth

@saraedum
Copy link
Member

comment:4

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?

@ohanar
Copy link
Member Author

ohanar commented Dec 19, 2011

fixed header

@ohanar

This comment has been minimized.

@ohanar
Copy link
Member Author

ohanar commented Dec 19, 2011

comment:5

Attachment: trac_12067.patch.gz

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.

@saraedum
Copy link
Member

Changed keywords from none to sd35

@jdemeyer
Copy link
Contributor

Merged: sage-4.8.alpha6

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

No branches or pull requests

3 participants