Closed jeffreyhanson closed 1 year ago
Merging #57 (322dd8b) into master (40e8e79) will decrease coverage by
0.27%
. The diff coverage is95.74%
.:exclamation: Current head 322dd8b differs from pull request most recent head bb0dc01. Consider uploading reports for the commit bb0dc01 to get more accurate results
@@ Coverage Diff @@
## master #57 +/- ##
==========================================
- Coverage 97.75% 97.48% -0.28%
==========================================
Files 10 10
Lines 491 517 +26
==========================================
+ Hits 480 504 +24
- Misses 11 13 +2
Impacted Files | Coverage Δ | |
---|---|---|
R/wdpa_clean.R | 95.91% <ø> (ø) |
|
R/wdpa_url.R | 96.61% <93.93%> (-3.39%) |
:arrow_down: |
R/st_repair_geometry.R | 100.00% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
This PR aims to address reviewer comments made by @Jo-Schie as part of the submission to JOSS. I plan to merge this once @Jo-Schie has confirmed that they're happy these changes have addressed their concerns.