Closed ndeloof closed 4 years ago
The change here is to stop using a fork, not introducing dependency which would legitimately require to explain why it is introduced The explanation you're asking for is here https://github.com/docker/app/commit/8beb2f139853113b3389cc6eeb75ce7d6b27997e
Merging #749 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #749 +/- ##
=======================================
Coverage 68.59% 68.59%
=======================================
Files 63 63
Lines 3477 3477
=======================================
Hits 2385 2385
Misses 775 775
Partials 317 317
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 a57a93e...05a8a01. Read the comment docs.
- What I did Stop using a fork and switch to upstream repo
- How I did it dep
- How to verify it I didn't breack the build :) (this is an e2e test dependency)
- Description for the changelog
- A picture of a cute animal (not mandatory)![image](https://user-images.githubusercontent.com/132757/69030693-9aa03d00-09d8-11ea-93da-49325a019745.png)