Closed eliascarv closed 2 years ago
Thank you @eliascarv , do you think the logic in Select itself could be changed without too much code to accommodate the rename option? Or the types of arguments we accept make it difficult?
Merging #109 (5637efd) into master (14aec71) will decrease coverage by
0.11%
. The diff coverage is50.00%
.:exclamation: Current head 5637efd differs from pull request most recent head 0802e1f. Consider uploading reports for the commit 0802e1f to get more accurate results
@@ Coverage Diff @@
## master #109 +/- ##
==========================================
- Coverage 94.14% 94.02% -0.12%
==========================================
Files 26 27 +1
Lines 735 737 +2
==========================================
+ Hits 692 693 +1
- Misses 43 44 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/transforms.jl | 98.43% <ø> (ø) |
|
src/transforms/selectrename.jl | 50.00% <50.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Superseded by #110
closes #106