ActivityWatch / aw-core

Core library for ActivityWatch
Mozilla Public License 2.0
48 stars 46 forks source link

WIP: Migrate peewee users to sqlite #86

Closed xylix closed 4 years ago

xylix commented 4 years ago

We could notify users that sqlite is the only supported backend going forward, or we could migrate config files via a script.

@ErikBjare Do you have any input on which way we should approach making existing users migrate?

codecov[bot] commented 4 years ago

Codecov Report

Merging #86 into master will decrease coverage by 0.22%. The diff coverage is 20%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   90.78%   90.55%   -0.23%     
==========================================
  Files          34       34              
  Lines        1606     1610       +4     
  Branches      249      250       +1     
==========================================
  Hits         1458     1458              
- Misses        113      116       +3     
- Partials       35       36       +1
Impacted Files Coverage Δ
aw_datastore/migration.py 63.15% <20%> (-7.44%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f43bbf4...fb1506a. Read the comment docs.

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging 9c9fd0096951bd586c15409922e9314aaf6ed5e5 into f43bbf47d2f192e1d104babff94e2aa8f9f5f226 - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 4 years ago

This pull request fixes 1 alert when merging fb1506abfb194714ea2a0bf56c8086b6dae7cec3 into f43bbf47d2f192e1d104babff94e2aa8f9f5f226 - view on LGTM.com

fixed alerts:

xylix commented 4 years ago

Closed because we're moving most users to aw-server-rust soon enough that this won't be necessary