open-telemetry / open-telemetry/weaver
Revisit what `titleCase` filter does
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 486
- Forks
- 108
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 40
Description
titleCase filter today capitalizes first letter and lowers everything after. E.g. it'd change dynamoDB to Dynamodb. I'm not sure if this behavior is useful in the context of semconv. If it it, someone can do str | lower before applying title case.
I think it should only capitalize and should not lower (i.e. produce DynamoDB). Here's original issue with the workaround. https://github.com/open-telemetry/weaver/issues/415#issuecomment-2504458374
If we believe titleCase has some use in its current shape, we can consider adding another filter that would have a different behavior
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the titleCase filter and reviewing how it is used in semconv, then read the linked original issue and workaround. Decide whether the filter should preserve existing capitalization or whether a separate behavior is needed; done means the chosen semantics are implemented and verified for inputs such as dynamoDB.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100