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

ref(core): Rename release management plugin name #647

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented Jan 8, 2025

  • Renames the release management plugin name from sentry-debug-id-upload-plugin to sentry-release-management-plugin
  • adds simple tests to catch renamings/regressions since they have a potential effect on SDKs like @sentry/sveltekit

Copy link
Member

@lforst lforst left a comment

Choose a reason for hiding this comment

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

Interesting PR description 😄 - otherwise lgtm

@Lms24 Lms24 self-assigned this Jan 8, 2025
@Lms24 Lms24 enabled auto-merge (squash) January 8, 2025 11:02
@Lms24 Lms24 merged commit d051b07 into main Jan 8, 2025
18 checks passed
@Lms24 Lms24 deleted the lms/fix-release-management-plugin-name branch January 8, 2025 11:04
Lms24 added a commit to getsentry/sentry-javascript that referenced this pull request Jan 9, 2025
…ps upload (#14942)

- replace the original release management plugin with a custom one
where we start release creation on `closeBundle`
- replace the original file deletion plugin with a custom one where we
start deletion on `closeBundle`
- in both cases, further ensuresthat the plugin is only invoked for
build and as late as possible (`enforce: 'post'`). 
- searche for the release management plugin by its old and new name
introduced in
getsentry/sentry-javascript-bundler-plugins#647
- slightly rename the custom source maps uploading plugin for better
convention; as well as some variables for better readability
- add unit tests for the new custom sub plugins
Lms24 added a commit to getsentry/sentry-javascript that referenced this pull request Jan 9, 2025
…ps upload (#14942)

- replace the original release management plugin with a custom one
where we start release creation on `closeBundle`
- replace the original file deletion plugin with a custom one where we
start deletion on `closeBundle`
- in both cases, further ensuresthat the plugin is only invoked for
build and as late as possible (`enforce: 'post'`). 
- searche for the release management plugin by its old and new name
introduced in
getsentry/sentry-javascript-bundler-plugins#647
- slightly rename the custom source maps uploading plugin for better
convention; as well as some variables for better readability
- add unit tests for the new custom sub plugins
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