-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Short form printing #144
Short form printing #144
Conversation
Codecov Report
@@ Coverage Diff @@
## master #144 +/- ##
==========================================
+ Coverage 95.69% 96.09% +0.40%
==========================================
Files 4 4
Lines 534 589 +55
==========================================
+ Hits 511 566 +55
Misses 23 23
Continue to review full report at Codecov.
|
can you get the coverage up? |
Please bump the version too so I can tag...Do we consider this a breaking a change? |
Your call, but I made it a patch for now. Some chance it breaks someone's doctests, I guess, but once they fix them they can bound the one version used for testing that. No functional change, and tagging a breaking change does impose some hassle on the other 90% of packages who don't have a doctest which prints this. |
Great, if no one raises an objection I'll merge and tag tomorrow. Thanks! |
I think the overloaded |
Sure, last commit is v0.12 |
Thanks! Maybe #147 can share the version bump too? |
This adds a shorter printing for when a FillArray appears inside a tuple, or in an array.
Before:
After: