-
Notifications
You must be signed in to change notification settings - Fork 23
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
Does not handle CallExpression in ExportDefaultDeclaration node #77
Does not handle CallExpression in ExportDefaultDeclaration node #77
Comments
I haven't seen one before! Can you try applying this patch which combines a load of outstanding PRs and see if that fixes it? |
Tried it, it doesn't work. It only patches |
I've tried reproducing this with a simple |
The issue only happens when |
resolves #12242 (although there are still some follow ups) https://github.com/open-telemetry/opentelemetry-js/releases/tag/v1.25.0 I think this lockfile looks correct, but lmk if this feels off. resolves #12011 resolves #12059 resolves #12154 resolves #12237 resolves nodejs/import-in-the-middle#77 cc @mohd-akram
resolves #12242 (although there are still some follow ups) https://github.com/open-telemetry/opentelemetry-js/releases/tag/v1.25.0 I think this lockfile looks correct, but lmk if this feels off. resolves #12011 resolves #12059 resolves #12154 resolves #12237 resolves nodejs/import-in-the-middle#77 cc @mohd-akram
Expected Behavior
Works.
Actual Behavior
Steps to Reproduce the Problem
I'm getting this after updating Sentry to 8.x. I'm not sure where exactly
import-in-the-middle
is being called from, but the gist of it is that it is crashing on this file (node_modules/adminjs/lib/frontend/components/app/sort-link.js
) because it has a function call in the export.require('import-in-the-middle/lib/get-esm-exports')({moduleSource: 'export default parseInt("1");', defaultAs: 'SortLink'})
Specifications
The text was updated successfully, but these errors were encountered: