Skip to content
This repository has been archived by the owner on May 8, 2021. It is now read-only.

Code cleanup for Python, JS #108

Merged
merged 12 commits into from
Jul 9, 2017
Merged

Code cleanup for Python, JS #108

merged 12 commits into from
Jul 9, 2017

Conversation

sumnerevans
Copy link
Owner

@sumnerevans sumnerevans commented Jul 8, 2017

Addresses some of #63 and all of #66. Cleaned up all of the Python and JS code.

@sumnerevans sumnerevans self-assigned this Jul 8, 2017
@codecov-io
Copy link

codecov-io commented Jul 8, 2017

Codecov Report

Merging #108 into master will decrease coverage by 0.05%.
The diff coverage is 56.93%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   41.02%   40.97%   -0.06%     
==========================================
  Files          39       43       +4     
  Lines        3054     3109      +55     
  Branches        1        1              
==========================================
+ Hits         1253     1274      +21     
- Misses       1801     1835      +34
Impacted Files Coverage Δ
server/widb_server.py 0% <ø> (ø) ⬆️
server/helpers/util.py 100% <ø> (ø) ⬆️
server/helpers/kajiki_view.py 60% <ø> (ø)
server/controller/sessions.py 27.02% <0%> (ø) ⬆️
server/parsing_lib/android_log_parser.py 0% <0%> (ø)
server/parsing_lib/ios_log_parser.py 0% <0%> (ø)
server/controller/root.py 54.34% <100%> (+8.4%) ⬆️
...r/datastore_interfaces/base_datastore_interface.py 100% <100%> (ø) ⬆️
server/tests/base_datastore_interface_tests.py 100% <100%> (ø) ⬆️
...ver/datastore_interfaces/no_datastore_interface.py 100% <100%> (ø) ⬆️
... and 15 more

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 5859f45...b60afbd. Read the comment docs.

@sumnerevans sumnerevans merged commit 9eb4f28 into master Jul 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants