open-telemetry / open-telemetry/weaver
Replace the regex in `from_semconv_specs` with the `https://docs.rs/semver/latest/semver/` crate.
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 486
- Forks
- 108
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 40
Description
In the weaver_semconv/src/registry.rs method from_semconv_specs (see todo), we should use: https://docs.rs/semver/latest/semver/ and URL parser that can give us the last element of the path to send to the parser.
See this PR comment: https://github.com/open-telemetry/weaver/pull/400/#discussion_r1905412697
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 in weaver_semconv/src/registry.rs at the from_semconv_specs method and its TODO. Read the semver crate documentation and the linked PR discussion, then inspect how the URL path is currently handled. Done means the regex is replaced with semver parsing using the final URL path element, with existing behavior preserved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100