-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
rfc(feature): Sentry Bundler Plugins API #86
Conversation
db5ff59
to
190b378
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to add notes about Node versions? I thought unplugin is Node 14+, which may cause issues.
Should we add an option to delete sourcemaps after upload? |
Co-authored-by: Lukas Stracke <[email protected]>
Not in this RFC. We will add a note in the changelog + migration guide. |
Ok, I think everything has been addressed. @Lms24 There is one open point about the rewriting - lmk if you think this is blocking, otherwise can anybody slap approve? cc @AbhiPrasad |
Nope, not blocking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📝
We wanna update our bundler plugins API with the next major.
Rendered RFC