Closed Myoldmopar closed 6 years ago
Totals | |
---|---|
Change from base Build 306: | 0.0% |
Covered Lines: | 1046 |
Relevant Lines: | 1046 |
Code Climate has analyzed commit 2b0662be and detected 0 issues on this pull request.
View more on Code Climate.
Merging #126 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #126 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 12 12
Lines 880 933 +53
Branches 20 20
=====================================
+ Hits 880 933 +53
Impacted Files | Coverage Δ | |
---|---|---|
recipes/static/recipes/js/services/recipes.js | 100% <ø> (ø) |
:arrow_up: |
.../tests/static/recipes/js/services/calendar.spec.js | 100% <100%> (ø) |
:arrow_up: |
recipes/static/recipes/js/controllers/planner.js | 100% <100%> (ø) |
:arrow_up: |
...ests/static/recipes/js/controllers/planner.spec.js | 100% <100%> (ø) |
:arrow_up: |
...s/tests/static/recipes/js/services/recipes.spec.js | 100% <100%> (ø) |
: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 ff4f58a...2b0662b. Read the comment docs.
Fixes a bunch of random things assigned to version 0.9.
125 Angular mocks is a dev dependency
122 Fix spacing in doc strings
121 Remove status from API responses
113 Update JS deps
110 Quick add duplicate recipe title now throws alert
To still do on this PR:
#111 Reapply filter text after adding recipe127 Test invalid HTTP responses for services