Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.6k
Add pretty printing for splat(f)#42717
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Failures are Distributed related (EAGAIN: Resource temporarily not available), seems like some tasks can't spawn their |
What if we printed it like |
Seelengrab commentedOct 20, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Well it'd be lying to you, as that's not how its implemented (edit: in this PR) 🤷 That used to be the old implementation though. Changing it was necessary to allow for easier printing. Compared to the old way of just printing whatever number anonymous function the result of Since it's not exported, I don't think it's used very wildly right now. I'm working on a package (PropCheck.jl) where using |
I'm talking about this PR 😅 In this PR, this is no longer a simple anonymous function, precisely because it makes printing easier. If it were not for the new struct, I'd have to call |
I should note that this works the exact same way as the anonymous function, except doing the hoisting to a |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
string can be overloaded, though it is uncommon, and usually just for performance to give the same result |
I think you meant to answer in the review thread above ;) |
I've incorporated the feedback, fixed printing, added the deprecation and replaced uses of the old syntax in the rest of Base. It's also rebased to the current master. Now let's hope nothing broke 😬 |
Seelengrab commentedMar 9, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Rebase & rerun of CI - I've also moved the deprecation notice to the 1.9 deprecation block, since this definitely isn't going to make 1.8 anymore. I'll write NEWS.md once CI is happy. |
Uh oh!
There was an error while loading.Please reload this page.
Add export? |
Uuh.. Unrelated? Doesn't look good though.. In either case, I've also added a NEWS.md entry since CI was green last night. Should be good to go now. |
The macos failure seems to stem from
|
I don't know where this should fall in the release cycle, but it'd be nice to get this in a future 1.9 :) |
Rebase on |
Failure in BuildKite macOS is |
When debugging calls with different invocations of
splat
, this helps to identify which function was being splatted into.master:
This PR:
Should
splat
be exported? It's pretty useful.