-
Notifications
You must be signed in to change notification settings - Fork 37
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
fix!: change the default delimiter of $str$concat
from "-"
to ""
and ignore_nulls
should be a named argument
#853
Conversation
I will update after merging #852 |
1a3b230
to
b9e7e97
Compare
$str$concat
from "-"
to ""
$str$concat
from "-"
to ""
$str$concat
from "-"
to ""
$str$concat
from "-"
to ""
and ignore_nulls
should be a named argument
- `<Expr>$str$concat()` method's `delimiter` argument's default value is | ||
changed from `"-"` to `""` (#853). |
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.
Should this be deprecated instead?
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.
I would like to change now as it is too much of a hassle to split the update into two separate updates.
…ignore_nulls` should be named argument
Co-authored-by: Etienne Bacher <52219252+etiennebacher@users.noreply.github.com>
ab2e880
to
0d3ba75
Compare
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.
Thanks!
Ref pola-rs/polars#13690