Skip to content

feat: Make opentelemetry optional - #566

Open
popzxc wants to merge 1 commit into
google:mainfrom
popzxc:optional-otel
Open

popzxc wants to merge 1 commit into
google:mainfrom
popzxc:optional-otel

Conversation

@popzxc

@popzxc popzxc commented Sep 17, 2026

Copy link
Copy Markdown

Fixes #564

Not all projects require opentelemetry, and besides being extra code to compile, it also makes supply chain security harder (see example).

I've kept opentelemetry enabled by default so that the change is backward compatible, hope that works.

@popzxc

popzxc commented Sep 22, 2026

Copy link
Copy Markdown
Author

@tikue any chance you can look into it?

@tikue

tikue commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Hi, thanks for the contribution! I will try to look at this in the next week or so.

@popzxc

popzxc commented Oct 12, 2026

Copy link
Copy Markdown
Author

@tikue sorry for pinging, would really appreciate if you'd find a minute to review the PR :)

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.

Make opentelemetry optional

2 participants