#16216: Improve PEP8 compliance: replace "== None" by "is None"
-------------------------------------+-------------------------------------
Reporter: wluebbe | Owner:
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.3
Component: build | Resolution:
Keywords: pep8 | Merged in:
Authors: Wilfried Luebbe | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/wluebbe/ticket/16216b | 6a2e8067c481abe8b5230dd1be3d86a9d9aa3d85
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by wluebbe):
* status: needs_work => needs_review
* commit: 1219d56b731119122356714ed4d26ae33c63cd5d =>
6a2e8067c481abe8b5230dd1be3d86a9d9aa3d85
* branch: u/wluebbe/ticket/16216 => u/wluebbe/ticket/16216b
Comment:
I pushed a fresh start based on Sage 6.2.
From commit message of the manual changes:
{{{
* changed "not xxx is None" to "xxx is not None" (where appropriate)
* reverted changes in docstring text
- usually did not revert changes to doctest *code*
- but reverted changes to expected results (if required)
* (usually) did not revert changes to *code* in comments
Always tried to apply good judgement - but refrained from changes
unrelated to the subject "is None".
}}}
----
New commits:
||[http://git.sagemath.org/sage.git/commit/?id=f623de2532c3c54b97d8a3deeb26e79a7bc6d945
f623de2]||{{{generated changes from "== None" to "is None"}}}||
||[http://git.sagemath.org/sage.git/commit/?id=6a2e8067c481abe8b5230dd1be3d86a9d9aa3d85
6a2e806]||{{{manual fixup of the generated changes}}}||
--
Ticket URL: <http://trac.sagemath.org/ticket/16216#comment:26>
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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
