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

DiagLoggerOptions interface is not exported #3638

Closed
altinokdarici opened this issue Feb 28, 2023 · 0 comments · Fixed by #3639
Closed

DiagLoggerOptions interface is not exported #3638

altinokdarici opened this issue Feb 28, 2023 · 0 comments · Fixed by #3639
Labels
bug Something isn't working priority:p4 Bugs and spec inconsistencies which do not fall into a higher prioritization

Comments

@altinokdarici
Copy link
Contributor

altinokdarici commented Feb 28, 2023

What happened?

Steps to Reproduce

import { DiagLoggerOptions } from '@opentelemetry/api'

Expected Result

Being able to import DiagLoggerOptions as it's a parameter type of DiagLoggerApi.setLogger

Actual Result

Module '"@opentelemetry/api"' has no exported member 'DiagLoggerOptions'.

Additional Details

DiagLoggerApi.setLogger has two overrides. One of them has options: DiagLoggerOptions parameter. However, DiagLoggerOptions is not exported through index.d.ts.

OpenTelemetry Setup Code

// N/A

package.json

No response

Relevant log output

No response

@altinokdarici altinokdarici added bug Something isn't working triage labels Feb 28, 2023
@pichlermarc pichlermarc added priority:p4 Bugs and spec inconsistencies which do not fall into a higher prioritization and removed triage labels Mar 1, 2023
@altinokdarici altinokdarici changed the title LoggerOptions interface is not exported DiagLoggerOptions interface is not exported Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:p4 Bugs and spec inconsistencies which do not fall into a higher prioritization
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants