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

Re-exported version variable causes an issue in production build #4217

Closed
jdbadilla opened this issue Oct 20, 2023 · 1 comment · Fixed by #4225
Closed

Re-exported version variable causes an issue in production build #4217

jdbadilla opened this issue Oct 20, 2023 · 1 comment · Fixed by #4225
Assignees
Labels
bug Something isn't working priority:p1 Bugs which cause problems in end-user applications such as crashes, data inconsistencies, etc

Comments

@jdbadilla
Copy link

jdbadilla commented Oct 20, 2023

What happened?

Steps to Reproduce

Due to confidentiality reasons, I cannot share the codebase with you. However, we have issues stemming from this second export * from './version' statement. Notice it is already defined previously here.

We are using webpack 4.16.1 and typescript 4.3.5 in our project. It would be awesome if you could push a fix removing this second unnecessary export statement so our minified build stops failing. We've invested several days into trying to identify the issue, and it seems that removing it may be the way to get our project to compile without having to make massive changes to our dependencies.

We are getting the following error due to the second import statement:
Cannot redefine property: VERSION

Expected Result

There should be no opentelemetry package that unnecessarily exports the same thing twice.

Actual Result

Opentelemetry currently star exports from the same file twice.

OpenTelemetry Setup Code

No response

package.json

No response

Relevant log output

No response

@jdbadilla jdbadilla added bug Something isn't working triage labels Oct 20, 2023
@dyladan dyladan added priority:p1 Bugs which cause problems in end-user applications such as crashes, data inconsistencies, etc and removed triage labels Oct 25, 2023
@david-luna
Copy link
Contributor

@dyladan you can assign this one to me :)

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:p1 Bugs which cause problems in end-user applications such as crashes, data inconsistencies, etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants