-
Notifications
You must be signed in to change notification settings - Fork 120
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
builtin function replace
does not support positional patterns
#347
Comments
catull
changed the title
builtin function
builtin function May 23, 2024
replace
deos not support positional patternsreplace
does not support positional patterns
This issue can be seen as an extension of #346, I realize it now. |
catull
pushed a commit
to catull/jslt
that referenced
this issue
May 26, 2024
This PR addresses issues schibsted#346 and schibsted#347.
catull
pushed a commit
to catull/jslt
that referenced
this issue
May 26, 2024
This PR addresses issues schibsted#346 and schibsted#347. Added documentation.
catull
pushed a commit
to catull/jslt
that referenced
this issue
May 26, 2024
This PR addresses issues schibsted#346 and schibsted#347. Added documentation.
catull
pushed a commit
to catull/jslt
that referenced
this issue
May 26, 2024
This PR addresses issues schibsted#346 and schibsted#347. Added support for Regexp pattern "Predefined character classes".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In issue #340 I had to resort to 2 nested
replace
calls.I wanted to collapse the two nested calls into one.
Both patterns start with
,
, one ends with}
, while the other ends in]
.It would be great if this was supported:
This way,
,}
would result in}
, and,]
would become ']'.Or think of the eternal battle of date formats, ISO vs US format vs European.
I am not going to suggest that this be supported in this issue:
The text was updated successfully, but these errors were encountered: