-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 text to support yielding lines, paragraphs or documents #3442
Conversation
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.
Awesome !
The parameter can also be named |
@lhoestq @mariosasko I would avoid the term
Please note that in the documentation, one of the terms more frequently used in this context is "row":
Other of the terms more frequently used in the docs (in the code as well) is "example":
Less frequently used: "item":
Other term used in the docs (although less frequently) is "sample". The advantage of this word is that it is also a verb, so we can use the parameter: "sample_by" (if you insist on using a verb instead of a noun). In summary, these proposals:
|
I like It could also be Not a big fan of the proposed nouns alone since they are very generic, that's why I tried to have something more specific. I also agree that we actually should avoid |
Thanks for the analysis of the used terms. I also like |
Thank you !! :D |
Add
config.row
option totext
module to allow yielding lines (default, current case), paragraphs or documents.Feel free to comment on the name of the config parameter
row
:example
,item