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

otl http exporter setup #89

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from
Draft

otl http exporter setup #89

wants to merge 14 commits into from

Conversation

RubLo
Copy link

@RubLo RubLo commented Apr 10, 2024

Context/Change

Please include a summary of the change and the reasoning and/or motivation behind it. List any dependencies that are
required and any potential issues to look out for.

Checklist

  • Ensure you have written good tests ❤️
  • Request reviewers (they would love to see your code 👮‍♂️)
  • Use labels 🏷 (if applicable)
  • Test your code in appropriate environment i.e development/staging/production

Reference

XXX-123

Comment on lines +123 to +131
if (isServer) {
// Module not found fix @see https://github.com/boxyhq/jackson/issues/1535#issuecomment-1704381612
config.plugins.push(
new webpack.IgnorePlugin({
resourceRegExp:
/(^@google-cloud\/spanner|^@mongodb-js\/zstd|^@sap\/hana-client\/extension\/Stream$|^@sap\/hana-client|^@sap\/hana-client$|^aws-crt|^aws4$|^better-sqlite3$|^bson-ext$|^cardinal$|^cloudflare:sockets$|^hdb-pool$|^ioredis$|^kerberos$|^mongodb-client-encryption$|^mysql$|^oracledb$|^pg-native$|^pg-query-stream$|^react-native-sqlite-storage$|^snappy\/package\.json$|^snappy$|^sql.js$|^sqlite3$|^typeorm-aurora-data-api-driver$)/,
})
);
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tried it without this if block and it worked too.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay I clean up all necessary stuff as well.
Also, need to make the PR green

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants