Closed mjordan closed 6 years ago
Merging #435 into master will decrease coverage by
0.01%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #435 +/- ##
==========================================
- Coverage 27.22% 27.21% -0.02%
==========================================
Files 79 79
Lines 5087 5090 +3
==========================================
Hits 1385 1385
- Misses 3702 3705 +3
Impacted Files | Coverage Δ | |
---|---|---|
src/metadatamanipulators/SplitRepeatedValues.php | 70.11% <0%> (-2.51%) |
:arrow_down: |
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 802f5bc...a9afb8d. Read the comment docs.
@jpeak5 @flummingbird If you would please review the LSU mentions, that would be appreciated. (Thumbs up or down would work well.) I'm OK with merging this since Travis CI passed. We can review the codecov/project failure separately. Thanks @mjordan for the PR.
Sorry @MarcusBarnes I didn't know a simple change to a README would trigger a codecov failure. 😞
@jpeak5 you have any problem with us merging this?
@mjordan looks great, thx!
Github issue: (#434)
What does this Pull Request do?
Adds links to some collections from the The Louisiana Digital Library migrated using MIK to the main README.md file.
How should this be tested?
n/a
Interested parties
@MarcusBarnes @jpeak5 @flummingbird