#20796: py3 print : not tested cases, step 3
-------------------------+-------------------------------------------------
Reporter: | Owner:
chapoton | Status: needs_work
Type: | Milestone: sage-7.3
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Jori Mäntysalo
python3 | Work issues:
Keywords: | Commit:
Authors: | ed93adc206ba2ba679d57cfb3bc461f7506f38b6
Frédéric Chapoton | Stopgaps:
Report Upstream: N/A |
Branch: |
public/20796 |
Dependencies: |
-------------------------+-------------------------------------------------
Changes (by jmantysalo):
* status: needs_review => needs_work
Comment:
What is the meaning of a space at start on
{{{
print(' {:>{width}}'.format(c, width=width), end="")
}}}
You have changed `while len(current_layer) > 0:` to `while
len(current_layer):`. It is OK, but isn't the Python way just `while
current_layer:`?
IMO there should be a dot after "Install database_gap first. See optional
packages".
There is an `'` at "Gel'fand", I don't know why.
Tests passed on modified files.
--
Ticket URL: <http://trac.sagemath.org/ticket/20796#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.