search

UAlbertaALTLab / morphodict

The Language Independent Intelligent Dictionary
https://morphodict.readthedocs.io/
Apache License 2.0
23 stars 11 forks source link

Use logic to check mobile db status #999

Closed andrewdotn closed 3 years ago

andrewdotn commented 3 years ago

Instead of relying on people remembering to update the CURRENT_DB_VERSION flag, which would have needed to become language-specific, use SQL queries and logic to determine if the bundled DB needs to copied into place.

codecov-commenter commented 3 years ago

Codecov Report

Merging #999 (9449268) into main (ab9d093) will decrease coverage by 0.45%. The diff coverage is 25.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #999      +/-   ##
==========================================
- Coverage   79.16%   78.71%   -0.46%     
==========================================
  Files         146      147       +1     
  Lines        5156     5202      +46     
  Branches      665      670       +5     
==========================================
+ Hits         4082     4095      +13     
- Misses        947      981      +34     
+ Partials      127      126       -1
Impacted Files Coverage Δ
src/mobile.py 0.00% <0.00%> (ø)
...dict/lexicon/management/commands/importjsondict.py 92.03% <100.00%> (+0.19%) :arrow_up:
...phodict/lexicon/migrations/0006_add_importstamp.py 100.00% <100.00%> (ø)
src/morphodict/lexicon/models.py 96.11% <100.00%> (+0.07%) :arrow_up:
src/CreeDictionary/CreeDictionary/views.py 79.62% <0.00%> (+0.92%) :arrow_up:

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 ab9d093...9449268. Read the comment docs.