cleanup of free Zinbiel algebras

[fchapoton]

full pep8, pyflakes and coverage

CC: @tscrim

Component: combinatorics

Author: Frédéric Chapoton

Branch/Commit: b2d12bc

Reviewer: Travis Scrimshaw

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

[fchapoton]

Commit: e8705b6

[fchapoton]

Branch: u/chapoton/25880

[fchapoton]

New commits:

e8705b6

fixing details in zinbiel algebras : full pep8, pyflakes and coverage

[tscrim]

comment:2

A while-we-are-at-it: can you change the header to use the standard copyright?

Bikeshedding: I find it better to break the 80 char/line guideline for code to avoid small breaks like this:

-        return super(FreeZinbielAlgebra, cls).__classcall__(cls, R, n, tuple(names))
+        return super(FreeZinbielAlgebra, cls).__classcall__(cls, R, n,
+                                                            tuple(names))

Bikeshedding nitpick:

-        - ``x`` and ``y`` -- two words
+        - ``x``, ``y`` -- two words

If you disagree with my comments, then I will be happy to set this to a positive review.

[tscrim]

Reviewer: Travis Scrimshaw

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

b2d12bc

some suggested details

[sagetrac-git]

Changed commit from e8705b6 to b2d12bc

[fchapoton]

comment:4

Thanks for the review, Travis. All done, I hope.

[tscrim]

comment:5

Thank you. LGTM.

[vbraun]

Changed branch from u/chapoton/25880 to b2d12bc