Closed finnhodgkin closed 5 years ago
Merging #99 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #99 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 5 5
Lines 230 230
=====================================
Hits 230 230
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 bf0aef1...ba7b80a. Read the comment docs.
Merging #99 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #99 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 5 5
Lines 230 230
=====================================
Hits 230 230
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 bf0aef1...96fff16. Read the comment docs.
@finnhodgkin only just saw this. 😮 (if PR isn't assigned to me then I don't get a notification for it...) 🔕 😞
:exclamation: No coverage uploaded for pull request base (
master@bf0aef1
). Click here to learn what that means. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #99 +/- ##
=======================================
Coverage ? 100%
=======================================
Files ? 5
Lines ? 230
Branches ? 0
=======================================
Hits ? 230
Misses ? 0
Partials ? 0
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 bf0aef1...2893433. Read the comment docs.
Probably not ready for merge because the advice on permissions for S3 is a little dubious... Currently we just open it to anyone and everyone
Adds a section on Amazon Simple Email Service
Relates to #97 #98