furmada / PythonOS

A touchscreen interface, library, and app platform for Python and Raspberry Pi.
MIT License
66 stars 14 forks source link

More modular PythonOS #28

Open NeoBelerophon opened 7 years ago

NeoBelerophon commented 7 years ago

Hi, first thing to say I like the idea of PythonOS very much. I thought about adding some apps for my radio project. Then I had the idea to add a gui component for it and was a bit frighten of that monolithic 2k loc block. So I started to break it down in a more modular way. a first draft can be found here: https://github.com/NeoBelerophon/PythonOS/tree/WIP_refactoring I would like to here what you think about it. IMHO this also could make portion to Python 3 easier and lower the entry barrier for other developers.

BR Neo

ghost commented 7 years ago

This is a great idea. PythonOS need to be more modular, and have a good range of testing coverage on non-GUI code too. We don't know what will happen if somebody successfully merged their pull request to the main repository, and suddenly introduced a bug that can very hard to track down later.

The Travis's tests are not actually testing smaller chunks of the codebase. What it does now, is only syntax checking. (which can be useless at certain point)