Skip to content

Conversation

RiotRampRoy
Copy link
Collaborator

Win64 was the only platform to have the "Warning" at the start of the message. Removing this brings it in line with messaging for the other platforms.

Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Removed the text "Warning" for Win64 from uplugin file ([#1084](https://github.com/getsentry/sentry-unreal/pull/1084))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against 2836e9c

Copy link
Collaborator

@tustanivsky tustanivsky left a comment

Choose a reason for hiding this comment

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

Since we no longer have a separate FAB (UE Marketplace) plugin package without symbol upload scripts the check here can be removed entirely. However, since some of the related changes are already made in #1076 it’s fine to just update the message for now.

@tustanivsky tustanivsky merged commit 3036e77 into main Sep 23, 2025
27 of 29 checks passed
@tustanivsky tustanivsky deleted the remove-warning-text-from-uplugin-file branch September 23, 2025 06:18
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