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

Docstrings and doctests for permgroup.py #2850

Closed
cswiercz mannequin opened this issue Apr 7, 2008 · 3 comments
Closed

Docstrings and doctests for permgroup.py #2850

cswiercz mannequin opened this issue Apr 7, 2008 · 3 comments

Comments

@cswiercz
Copy link
Mannequin

cswiercz mannequin commented Apr 7, 2008

Provide missing docstrings and doctests for groups/perm_gps/permgroup.py (CURRENT SCORE permgroup.py: 81% (59 of 72))

Component: documentation

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

@cswiercz cswiercz mannequin added this to the sage-3.0 milestone Apr 7, 2008
@cswiercz cswiercz mannequin added c: documentation labels Apr 7, 2008
@cswiercz cswiercz mannequin self-assigned this Apr 7, 2008
@cswiercz
Copy link
Mannequin Author

cswiercz mannequin commented Apr 13, 2008

comment:1

Attachment: groups.perm_gps.permgroup.patch.gz

FINAL SCORE permgroup.py: 98% (71 of 72)

@cswiercz cswiercz mannequin added the s: needs review label Apr 13, 2008
@mwhansen
Copy link
Contributor

comment:2

Attachment: 2850.patch.gz

2850.patch fixes an indentation error in calculus.py and should be the only patch applied.

Other than that, it looks good to me.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Apr 15, 2008

comment:3

Merged 2850.patch in Sage 3.0.alpha5

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Apr 15, 2008
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

1 participant