Skip to content

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Feb 10, 2026

This PR converts the webpack plugin to be a pure webpack plugin without using unplugin.

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Webpack no longer uses unplugin by timfish in #870

🤖 This preview updates automatically when you update the PR.

@timfish timfish marked this pull request as ready for review February 10, 2026 15:58
@timfish timfish requested review from Lms24 and chargome February 10, 2026 15:58
@timfish timfish changed the title feat: webpack no longer use unplugin feat: webpack no longer uses unplugin Feb 10, 2026
Copy link
Member

@chargome chargome left a comment

Choose a reason for hiding this comment

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

Generally LGTM, just left one question. Feel free to merge if I misunderstood this.

);

void sentryBuildPluginManager
.createRelease()
Copy link
Member

Choose a reason for hiding this comment

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

Don't we want to do this regardless of the sourcemapsEnabled check above?

Copy link
Collaborator Author

@timfish timfish Feb 11, 2026

Choose a reason for hiding this comment

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

More than likely yes? Seer picked this up on both the other PRs so I guess we should be.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

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