-
-
Notifications
You must be signed in to change notification settings - Fork 57
chore: Remove deprecated runtime/build time configuration #2337
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
chore: Remove deprecated runtime/build time configuration #2337
Conversation
<WindowsNativeSupportEnabled>k__BackingField: 1 | ||
<MacosNativeSupportEnabled>k__BackingField: 1 | ||
<LinuxNativeSupportEnabled>k__BackingField: 1 | ||
<XboxNativeSupportEnabled>k__BackingField: 1 |
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.
Missed checking this in to the samples when adding Xbox support.
AndroidNativeSupportEnabled = AndroidNativeSupportEnabled, | ||
NdkIntegrationEnabled = NdkIntegrationEnabled, | ||
NdkScopeSyncEnabled = NdkScopeSyncEnabled, | ||
PostGenerateGradleProjectCallbackOrder = PostGenerateGradleProjectCallbackOrder, |
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.
This was previously wrongly only on the scriptable object and not on the actual options, making them inaccessible programmatically.
Description
The changes in #1888 replaced the runtime/build time configuration setup into one
Option Configuration
setup. The same callback would get invoked wether it is on runtime or at build time.We kept the old setup around as to not break people when upgrading but it's been 2 mayor releases now.
A migration guide is already in place: https://docs.sentry.io/platforms/unity/migration/#changes-to-the-programmatic-configuration.