-
Notifications
You must be signed in to change notification settings - Fork 476
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
Support append() #2046
Support append() #2046
Conversation
addsuffix(suffix, base) takes a `suffix` and a (possibly space-separated) base string, and append `suffix` to the end of each word in the base string; for example, addsuffix('.c', 'foo bar baz') -> 'foo.c bar.c baz.c'
Seems reasonable to me! How about calling it |
Shorter names are okay too. This is carried over from when it was about porting make functions. I'll incorporate the suggestion changes. Thanks for your feedbacks |
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'm not just
maintainer, did you mean to request review from casey? Anyway this looks good to me, append()
now behaves according to make
documentation of addsuffix
as intended. 👍
Nice, LGTM! I added |
Pull request was closed
Gah, didn't mean to close. |
Got it! |
append(suffix, base)
takes aappend
and abase
string with space-separated words,and append
suffix
to the end of each word in the base string.For example,
append('.c', 'foo bar baz')
-> 'foo.c bar.c baz.c'