Skip to content
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

processor/strings: add support for title convertor #6982

Merged

Conversation

them0ntem
Copy link
Contributor

  • Title (https://golang.org/pkg/strings/#Title) function support all unicode letters.
  • BUG: The rule Title uses for word boundaries does not handle Unicode punctuation properly, that should not effect much with use-case of Title rule in telegraf.

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

- Title(https://golang.org/pkg/strings/#Title) function support all unicode letters.
- BUG: The rule Title uses for word boundaries does not handle
Unicode punctuation properly.
@danielnelson danielnelson added this to the 1.14.0 milestone Feb 5, 2020
@danielnelson danielnelson added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Feb 5, 2020
@danielnelson danielnelson merged commit ae22db4 into influxdata:master Feb 5, 2020
@danielnelson
Copy link
Contributor

Thanks, nice addition.

athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@them0ntem them0ntem deleted the feature/processor/strings branch May 12, 2020 17:59
jcgonnard pushed a commit to jcgonnard/telegraf that referenced this pull request Jun 11, 2020
idohalevi pushed a commit to idohalevi/telegraf that referenced this pull request Sep 29, 2020
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants