Closed nelsonic closed 2 years ago
Merging #146 (fa71157) into master (9de32c6) will not change coverage. The diff coverage is
n/a
.:exclamation: Current head fa71157 differs from pull request most recent head 7f9e2ff. Consider uploading reports for the commit 7f9e2ff to get more accurate results
@@ Coverage Diff @@
## master #146 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 3 3
=========================================
Hits 3 3
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 9de32c6...7f9e2ff. Read the comment docs.
Read
: /code-reuse-hexpm.md~~This PR is not "finished" but it already contains a lot of useful stuff. So I figure we should just merge this as-is and extend it when we have time.~~
I tidied it up and it's reviewable / mergeable in it's current state. π
@SimonLab this is totally not urgent. Defo focus on your work-work first. π
It will be useful to people learning Elixir
so keen to publish at some point. :shipit:
Thanks.
@SimonLab when you have T25m
please read through this and provide feedback. π
Thank you for reading it with a close eye and making all the typo-fixes, Simon! π π
The purpose of this PR is to show a real world example for re-using code ... #95 β»οΈ It's not "finished" but it already contains a lot of useful info so I think it's worth merging. π We can extend it when life is a bit less hectic. π€