Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback table (rebased ✅) #30

Merged
merged 6 commits into from
May 4, 2017
Merged

Feedback table (rebased ✅) #30

merged 6 commits into from
May 4, 2017

Conversation

jackcarlisle
Copy link
Member

@jackcarlisle jackcarlisle commented Apr 6, 2017

ready for review

@codecov-io
Copy link

codecov-io commented Apr 6, 2017

Codecov Report

Merging #30 into master will decrease coverage by 11.11%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #30       +/-   ##
===========================================
- Coverage     100%   88.88%   -11.12%     
===========================================
  Files           5        5               
  Lines          14        9        -5     
===========================================
- Hits           14        8        -6     
- Misses          0        1        +1
Impacted Files Coverage Δ
web/models/feedback.ex 50% <50%> (ø)
web/models/user.ex

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 ddc0bfb...ff89b8e. Read the comment docs.

@jackcarlisle jackcarlisle mentioned this pull request Apr 6, 2017
1 task
@jackcarlisle jackcarlisle requested a review from nelsonic May 3, 2017 13:05
@jackcarlisle jackcarlisle changed the title Feedback table Feedback table (rebased ✅) May 3, 2017
@jackcarlisle
Copy link
Member Author

@nelsonic my test coverage locally is 100%. The remote test coverage was 100% before the rebase as well but it's gone down to 87.5% now. Looking into why this is the case.

screen shot 2017-05-03 at 14 16 51

@jackcarlisle jackcarlisle removed the request for review from nelsonic May 3, 2017 13:19
@jackcarlisle jackcarlisle assigned jackcarlisle and unassigned nelsonic May 3, 2017
@nelsonic
Copy link
Member

nelsonic commented May 4, 2017

@jackcarlisle please assign for review when you think it's ready. 😉
I want to approve your timesheet/invoice but need to clear down the PR backlog first. 👍

@jackcarlisle
Copy link
Member Author

@nelsonic I've been looking into why the coverage has dropped and I think it will get back up to 100% after the next PR is merged. It's got something to do with not having any feedback controller tests which means that the model isn't hit. I think the next PR will solve this though so I'm going to assign this as ready

Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jackcarlisle do you want to add .env to .gitignore and reset the AWS secrets (ASAP) then re-assign to me for merging?

@nelsonic nelsonic removed their assignment May 4, 2017
@jackcarlisle
Copy link
Member Author

@nelsonic yeah sorry just saw that!

@jackcarlisle
Copy link
Member Author

@nelsonic secrets changed, ignored and deleted from AWS

@nelsonic
Copy link
Member

nelsonic commented May 4, 2017

@jackcarlisle cool. thanks for confirming. merging now. 👍
https://vimeo.com/50600813

@nelsonic nelsonic merged commit 3164801 into master May 4, 2017
@nelsonic nelsonic deleted the feedback-table branch May 4, 2017 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants