Skip to content

Use named OpenTelemetry semantic convention imports - #269

Merged
rbarbadillo merged 1 commit into
mainfrom
rbarbadillo/obs-2228-semconv-named-imports
Sep 9, 2026
Merged

rbarbadillo merged 1 commit into
mainfrom
rbarbadillo/obs-2228-semconv-named-imports

Conversation

@rbarbadillo

@rbarbadillo rbarbadillo commented Sep 8, 2026 •

Copy link
Copy Markdown
Member

Summary

This PR replaces the namespace import from @opentelemetry/semantic-conventions/incubating with named imports for the constants used by the observability tracing code.

This is a small bundle-size improvement for users who include the observability path. It should let bundlers drop unused semantic-convention constants instead of retaining the full constant table.

Checks

  • pnpm run build
  • pnpm run lint

@rbarbadillo rbarbadillo self-assigned this Sep 8, 2026
@rbarbadillo
rbarbadillo force-pushed the rbarbadillo/obs-2228-semconv-named-imports branch from e560eb8 to 09ce8df Compare September 8, 2026 22:20
@rbarbadillo
rbarbadillo force-pushed the rbarbadillo/obs-2228-semconv-named-imports branch from 09ce8df to 0c7bc6f Compare September 8, 2026 22:28
@rbarbadillo
rbarbadillo marked this pull request as ready for review September 8, 2026 22:33
@rbarbadillo
rbarbadillo merged commit f669ec8 into main Sep 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants