-
Notifications
You must be signed in to change notification settings - Fork 370
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
Extend stack
from Base/Compat
#3120
Conversation
Added to Julia in PR 43334, added to Compat in PR 777 (v4.2).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good apart from DataFrames.jl version. Should be merged after Compat.jl is released.
Co-authored-by: Bogumił Kamiński <bkamins@sgh.waw.pl>
Compat v4.2.0 is now merged to General so I closed and reopened to restart CI so that it will pick up that version. |
Co-authored-by: Bogumił Kamiński <bkamins@sgh.waw.pl>
Looks like the Windows failures on CI are OOMs. |
Thank you! |
CC @nalimilan |
Shouldn't we always export |
Indeed, it shouldn't make a difference since Base exports it. In this case, I just made it follow suit with how we're handling other conditional extensions; those aren't unconditionally listed among the exports. |
Added to Julia in JuliaLang/julia#43334, added to Compat in JuliaLang/Compat.jl#777 (v4.2). This PR requires the latter to be merged and tagged first.
Closes #3119.