search

emberobserver / client

http://emberobserver.com/
124 stars 65 forks source link

[WIP] Changes necessary in the frontend to support new score (decimal, with #145

Closed kategengler closed 5 years ago

kategengler commented 5 years ago

explanation from server)

WIP cause waiting on backend

codecov-io commented 5 years ago

Codecov Report

Merging #145 into master will decrease coverage by 0.25%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #145      +/-   ##
=========================================
- Coverage   79.05%   78.8%   -0.26%     
=========================================
  Files         102     106       +4     
  Lines         807     821      +14     
=========================================
+ Hits          638     647       +9     
- Misses        169     174       +5
Impacted Files Coverage Δ
app/models/addon.js 100% <ø> (ø) :arrow_up:
app/controllers/addons/show.js 100% <ø> (ø) :arrow_up:
app/helpers/score-component-color.js 100% <100%> (ø)
app/components/score-detail.js 100% <100%> (ø) :arrow_up:
app/helpers/format-score-contribution.js 100% <100%> (ø)
app/helpers/to-fixed.js 100% <100%> (ø)
app/components/score-display.js 100% <100%> (ø) :arrow_up:
app/helpers/check-pass-proportion.js 100% <100%> (ø)
app/models/review.js 28.57% <0%> (-71.43%) :arrow_down:
... and 3 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 ae117df...e89e8be. Read the comment docs.