Coala: no need for # +ELLIPSIS

Created on 20 Nov 2016  路  4Comments  路  Source: coala/coala

many doctests use # +ELLIPSIS to support ... in outputs. This seems to be default and therefore obsolete :)

difficultlow
Source
picture userzimmermann
🎉1

Most helpful comment

IMO remove it, it works with our test setup and if someone uses another test setup he can still configure it to be default.

picture sils on 17 Jan 2017
👍2

All 4 comments

how are you planning to remove it?
The documentation still strongly implies it is necessary.
As far as I am aware, doctest runner will fail unless it exists.

jayvdb picture jayvdb on 21 Nov 2016

Does not work, # +ELLIPSIS is not a default at least for our test setup. Is it possible to set some flag in setup.cfg?

Makman2 picture Makman2 on 21 Nov 2016

There's https://github.com/scikit-learn/scikit-learn/blob/master/setup.cfg#L23

adtac picture adtac on 24 Nov 2016
👍1

IMO remove it, it works with our test setup and if someone uses another test setup he can still configure it to be default.

sils picture sils on 17 Jan 2017
👍2
Was this page helpful?
0 / 5 - 0 ratings

Related issues

userzimmermann picture userzimmermann  路  5Comments
jayvdb picture jayvdb  路  6Comments
nvzard picture nvzard  路  5Comments
jschwarzwalder picture jschwarzwalder  路  6Comments
jayvdb picture jayvdb  路  6Comments