Closed naazy closed 7 years ago
Merging #151 into master will decrease coverage by
0.54%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #151 +/- ##
==========================================
- Coverage 98.32% 97.77% -0.55%
==========================================
Files 23 23
Lines 179 180 +1
==========================================
Hits 176 176
- Misses 3 4 +1
Impacted Files | Coverage Ξ | |
---|---|---|
web/controllers/rules/issue/time_estimation.ex | 100% <100%> (ΓΈ) |
:arrow_up: |
web/models/user.ex | 50% <0%> (-50%) |
: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 4d1b10f...957a772. Read the comment docs.
@naazy, hoorah! π It's review time! π
I couldn't help but notice that there isn't an in-progress
label on this pull request and a Reviewer
has been added...makes me think that this pull request is ready for review π€
To save you time β³ I've added the Reviewer as an Assignee and I've added the awaiting-review
label - automatically - just like magic! π© π° β¨. Please correct me if I'm wrong, but if I got it right
this time I hope it helps you! π
Any questions, complaints, feedback, contributions? If you prefer, you can also send us anonymous feedback: https://dwyl-feedback.herokuapp.com/feedback/new
Adding @danwhy and @shouston3 as potential reviewers as they've shown interest in dwylbot in the past. If you guys have some spare time during the workday, please take a look π
Have addressed your comments @shouston3. Thanks :smile:
epics do not need to be time estimated #138