Documentation preview for this PR (built with commit b90f0bb; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@dimpase: Should sage-devel be asked whether it is okay to change the gens of sage.modular.abvar.finite_subgroup.FiniteSubgroup from (immutable) Sequence to tuple? I'm worried that there might have been a reason to use Sequence, so the change might break some code. Although the output type is not stated in the docstring, it is in the type annotation. (Or maybe delay this one change to early in the next release cycle, so it gets tested?)

Also, two minor edits:

line 379 of src/sage/coding/linear_code_no_metric.py
- Return the generators of this code as a list of vectors.
+ Return the generators of this code as a tuple of vectors.

line 574 of src/sage/libs/eclib/interface.py
- Return a list of the generators for the Mordell-Weil group.
+ Return a tuple of the generators for the Mordell-Weil group.

@DaveWitteMorris Looking at the Sequence example again it does look like it's turning a the list B into an immutable sequence. So the only difference between Sequence and tuple would be if Sequence has more functionality than tuple.

@DaveWitteMorris I've change the documentation of gens in AbstractLinearCodeNoMetric, mwrank_EllipticCurve, and FiniteSubgroup to reference that they are returning tuples not lists.

@DaveWitteMorris If you could take a look at this PR again and see if it is ready to be set to positive review that would be great! I think we we're discussing if a tuple is equivalent to an immutable sequence.

I ran make ptestlong to confirm that the doctest errors are not coming from this PR.