-
Notifications
You must be signed in to change notification settings - Fork 2
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
CommonJS import error #13
Comments
@herzog0 Thanks for reporting this issue. It is originating from For now, we recommend downgrading |
Hi, this is a very recent issue I see :) I too am having the same issue, even though using I removed all my |
@mdahiemstra Can you run |
I am in the same situation. Below I have pasted the contents of yarn.lock with excerpts where necessary.
|
For "resolutions": {
"@opentelemetry/resources": "<=1.14.0",
"@opentelemetry/sdk-trace-node": "<=1.14.0",
"@opentelemetry/semantic-conventions": "<=1.14.0"
} For "overrides": {
"@opentelemetry/resources": "<=1.14.0",
"@opentelemetry/sdk-trace-node": "<=1.14.0",
"@opentelemetry/semantic-conventions": "<=1.14.0"
} This will make sure that all the opentelemetry dependencies that |
Thanks for presenting the code. I appreciate your support. |
Thanks, i can verify this works. Except the format for pnpm should be:
Confirmed working! |
You can also resolve this by using the
|
I'm trying to use the package in my project, but I'm getting an import error deep inside the @vercel/otel package, so there's nothing I can do about it.
I created the official nextjs example from the docs to ensure this wasn't a problem specific to my project.
NodeJS version: 18.16.0
npm version: 9.5.1
Update
Apparently, the problem was that I followed these official instructions and there is missing the need to install
@opentelemetry/api
, which causes the error above.Following other recipe solved the issue.
The text was updated successfully, but these errors were encountered: