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

feat(node): Rework ANR to use worker script via an integration #9945

Merged
merged 38 commits into from
Dec 21, 2023

Conversation

timfish
Copy link
Collaborator

@timfish timfish commented Dec 20, 2023

This is another attempt at #9823 since that broke build caching by writing to the source directory and had to be reverted.

This now runs the worker rollup build after the Node SDK transpile and overwrites the worker-script.js files in the build output.

@timfish timfish marked this pull request as ready for review December 20, 2023 23:17
@timfish timfish requested a review from lforst December 20, 2023 23:17
@lforst
Copy link
Member

lforst commented Dec 21, 2023

Thank you for redoing this work!

@lforst lforst merged commit 6173846 into getsentry:develop Dec 21, 2023
75 checks passed
@timfish timfish deleted the anr-worker-from-base64-script branch December 21, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants