search

limetext / backend

Backend for LimeText
BSD 2-Clause "Simplified" License
538 stars 87 forks source link

Refactored packages #84

Closed zoli closed 8 years ago

zoli commented 8 years ago

As a whole this pr is about #71 it doesn't resolve all of the issues but it gets us closer to modularity/optionality. Now the core backend doesn't know anything about sublime packages for using them in the frontend we just need to add 

import _ "github.com/limetext/lime/backend/sublime"

closes #64

As there are a lot of commits and changes I'll address the main changes:

  1. packages package which most of the code and the package is documented
  2. sublime package the code is now much cleaner also we have separate tests
  3. removed lime-packages dependency from backend we dont need this submodule for lime-backend anymore
zoli commented 8 years ago

I tested the frontends against this pr they won't crash but they don't load any packages currently because I need to work a bit on lime-packages repo.

zoli commented 8 years ago

closes #85

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.3%) to 58.421% when pulling 02d187f9e940a464f41a5f42e1b276fdee3933c7 on packages into 1f446ac54d08be96b16bda78c4c5ed167f45a052 on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.3%) to 58.396% when pulling cb459ef64edd9d2e5b41ddbad902b0bd0c28001b on packages into 1f446ac54d08be96b16bda78c4c5ed167f45a052 on master.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-1.0%) to 58.726% when pulling c3f7f27887e66183971933a2efb7b154177e06e9 on packages into 1f446ac54d08be96b16bda78c4c5ed167f45a052 on master.

zoli commented 8 years ago

All mentioned problems are resolved.