Hello folks, up to now the rules on tabs in Sage library sources has been "don't use them", but there was never any enforcement and/or systematic detection since no side effect of using tabs has even been detected. As it turns out #5653 exposes a case where tabs cause problems when rendering docstrings in a pretty way in the notebook.
At #5848 John Palmieri posted a patch converting all tabs in the Sage library to spaces. This kind of patch is likely to break some patches that used to work and it is generally not too nice to do "white space only" patches for exactly that reason. But this case is somewhat special since it will allow #5653 to work better and once you have seen that patch in action in the notebook you will likely agree that a little short term pain is well worth the potential rebasing problem. If this goes into 3.4.1.final it needs to be done soon, i.e. less than two hours. John is happy to rebase the patch at #5848, so it could be the last patch to go into 3.4.2, too, for example. Thoughts? Cheers, Michael --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to sage-devel-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---
