PR: Change variableexplorer viewer collections name to iterables

[rlaverde]

See: #5206 (comment)

In #4557 collectionseditor.py was changed to collections.py (and that interferes with collections built-in)

This have to be rebased after #5206

[pep8speaks]

Hello @rlaverde! Thanks for updating the PR.

• In the file spyder/plugins/variableexplorer/tests/test_iterablesviewer.py, following are the PEP8 issues :

Line 34:80: E501 line too long (100 > 79 characters)
Line 89:80: E501 line too long (100 > 79 characters)

• In the file spyder/plugins/variableexplorer/widgets/viewers/iterables.py, following are the PEP8 issues :

Line 456:9: E265 block comment should start with '# '
Line 528:9: E265 block comment should start with '# '
Line 1162:33: E127 continuation line over-indented for visual indent

Comment last updated on September 14, 2017 at 12:45 Hours UTC

[ccordoba12]

Please rebase this one. Also, please rename test_collectionseditor.py to test_iterablesviewer.py

[rlaverde]

Also array.py interferes, I'm not sure if also dataframe.py object.py or text.py intereferes too

[goanpeca]

Maybe we should add a prefix letter or something to each object?

sobject, sarray, sdataframe, scollection ? (for spyder?) or something similar

[rlaverde]

yes, or maybe v for viewer, or e for editor

[goanpeca]

Sure, lets use something like that and get rid of these problems for good. Maybe with underscore to make it clearer?

ve_object
ve_array
ve_collections

ve = variable explorer ?
etc...

@ccordoba12 ?

[ccordoba12]

Then we should remove the viewers module and leave things as they were before, i.e. collectionseditor.py, dataframeeditor.py, etc.

[rlaverde]

Closing in favor of #5263