-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
@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. |
@jackcarlisle please assign for review when you think it's ready. 😉 |
@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 |
There was a problem hiding this 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 yeah sorry just saw that! |
@nelsonic secrets changed, ignored and deleted from AWS |
@jackcarlisle cool. thanks for confirming. merging now. 👍 |
ready for review