-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
change parsing of ~
from macro to operator
#20406
Comments
Currently this has n-ary parsing; |
Regarding updating DataFrames, the situation is a little complex, but we're getting there. Ref JuliaData/DataFrames.jl#1154 (comment) |
@JeffBezanson I've never seen that used in the wild (and indeed I didn't even know about it) so I'd guess that if that behavior were removed with the parsing change, few people if anyone would notice. So if it's simpler to take it out then I'd say go for it. |
Good point: try parsing as binary and see if anyone screams 😬 |
For what it's worth I have made use of the n-ary parsing, but it's moot when the macro parsing goes away. |
fix #20406, parse infix `~` as a normal operator
From #19598:
change parsing of
~
in Base from macro to plain infix operator (fix #20406, parse infix~
as a normal operator #20420)change the one user of this as a macro – i.e. StatsModels (WIP: Replace infix ~ for formulas with a model macro JuliaStats/StatsModels.jl#9)
update DataFrames to depend on the new version of StatsModels (issue/PR link?)
The text was updated successfully, but these errors were encountered: