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

fix: Use magic-string appendLeft instead of replace #303

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

lforst
Copy link
Member

@lforst lforst commented Jun 6, 2023

This is an experimental fix that might resolve #298

I tried a bunch of things with our RegExp to understand whether it is wrong but couldn't reproduce the examined outcome.

This led me to believe that there is something going on inside the magic-string library that might cause the issue. This is an experimental fix that should be equivalent in behaviour but will hopefully resolve the issue.

@lforst lforst requested a review from mydea June 6, 2023 12:19
packages/bundler-plugin-core/src/index.ts Dismissed Show dismissed Hide dismissed
@lforst lforst merged commit b02821a into main Jun 6, 2023
16 checks passed
@lforst lforst deleted the lforst-attempt-fixing-faulty-injection-logic branch June 6, 2023 13:16
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.

ERROR: Unexpected ";" when using sentryVitePlugin
2 participants