remove dependancy on internal base func - #442
Conversation
|
But coverage is 100%? How is it that it can't be called? As the comment makes clear, this is to support downstream BlockArrays.jl |
|
sorry, like I was saying in my edit; I was going too fast. it does get called; but when looking at code only in your repo it just sends execution back down the normal display pipeline. this section should maybe be in wow, just realized you can run the origin's ci workflows from pull requests on your own fork... |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #442 +/- ##
==========================================
- Coverage 99.91% 99.91% -0.01%
==========================================
Files 9 9
Lines 1234 1232 -2
==========================================
- Hits 1233 1231 -2
Misses 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
it doesn't seem like it even changes anything, just calls an expensive
invokefor some reason? I don't know what this was trying to do but I'll fix it if it breaks anything somehow; trying to clean up these internal array print functions in base and found your package complaining that "print_matrix_rownot defined inBase" when importing a downstream packageedit: ok, I think I'm starting to see what this is trying to do. prob should have started at block arrays as mentioned in the comment...