Refactor sync - Githubissues

andreikop / enki

A text editor for programmers

http://enki-editor.org

GNU General Public License v2.0

161 stars 38 forks source link

Refactor sync #212

Closed bjones1 closed 10 years ago

bjones1 commented 10 years ago

Per suggestions in issue #17, refactor sync code from preview.py and test_preview.py into separate files.

On a side note, I don't know how to separate this pull request from the earlier request (codechat.png icon removal) -- what's your recommended process?

andreikop commented 10 years ago

Merged! 2 issues:

Is it possible to split preview sync one more on preview->code and code->preview sync? Does it make sense?
I have a lot of test failures, if tre is not installed. Bryan, could you try to test this?

bjones1 commented 10 years ago

For now, I think keeping all sync code together makes sense. I plan on some updates and improvements, where I may be able to combine pieces of code->web and web->code sync. I'll at least re-organize the code to make the two pieces more distinct.

I'll definitely look at / fix test failures produced when tre isn't installed.