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
added docs for prefix_number and prefix_street #2172
Conversation
@@ -221,6 +253,8 @@ _Example_ | |||
} | |||
``` | |||
|
|||
The source data should be examined to determine if the shorthand methods prefix_number and prefix_street could be used instead of regexp. |
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.
Can you add `'s around the function names for clarity.
@@ -147,7 +147,9 @@ This list gives a brief summary of what each function does. Examples can be foun | |||
|
|||
Function | Note | |||
-------- | ----- | |||
`regexp` | Allow regex find and/or replace on a given field. Useful to extract house number/street/city/region etc when the source has them in a single field | |||
`prefix_number` | Allow number to be extracted from a single field using an internal convenience regex. |
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.
A minor nit: do you think it would make sense to remove reference to it being a regular expression behind the scenes? This way we can swap out the backend implementation with something non-regex-ey later. Or maybe it doesn't matter...
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.
Yeah — maybe we should provide some example values here instead to show what kind of inputs it works for?
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.
This looks good to me.
No description provided.