Skip to content

Unity 7.0 requires now AutoStaticsCleanup/NoAutoStaticsCleanup for static variables - #228

Merged
todi1856 merged 7 commits into
masterfrom
static-cleanup-for-coreclr
Oct 22, 2026
Merged

todi1856 merged 7 commits into
masterfrom
static-cleanup-for-coreclr

Conversation

@todi1856

@todi1856 todi1856 commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Yamato CI

Fix

warning UAL0013: A type with static fields, automatic properties or events must have a Unity.Scripting.LifecycleManagement.AutoStaticsCleanup or NoAutoStaticsCleanup attribute. AutoStaticsCleanup additionally requires the type and all its containing types to be declared partial.

…or events must have a Unity.Scripting.LifecycleManagement.AutoStaticsCleanup or NoAutoStaticsCleanup attribute. AutoStaticsCleanup additionally requires the type and all its containing types to be declared partial. for Unity 7
@todi1856
todi1856 requested a review from BorisShir October 21, 2026 13:54

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

Needs changes

The change adds lifecycle-cleanup annotations and makes static values readonly or constant, but the compatibility handling breaks compilation on supported Unity versions.

Reviewed commit 29ef9a1

🤖 Helpful? 👍/👎

@todi1856

Copy link
Copy Markdown
Member Author

/review

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

Ship it

The change adds lifecycle cleanup opt-outs and makes static references readonly or constant. I examined compatibility guards, polyfill accessibility, field usages, and reference checks; I found no new actionable issues.

Reviewed commit ab462ea

🤖 Helpful? 👍/👎

todi1856 and others added 2 commits October 22, 2026 11:24
…coreclr

# Conflicts:
#	com.unity.mobile.android-logcat/Editor/AndroidLogcatScreenCaptureWindow.cs
#	com.unity.mobile.android-logcat/Editor/AndroidLogcatSettingsProvider.cs
#	com.unity.mobile.android-logcat/Editor/AndroidLogcatStyles.cs
#	com.unity.mobile.android-logcat/Tests/Editor/AndroidLogcatNetTests.cs
…ster

Fixes UAL0013 on Unity 7 for AndroidLogcatCaptureList.Styles,
AndroidLogcatImageViewer.Styles, AndroidLogcatLiveStream and
AndroidLogcatUtilities.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@todi1856
todi1856 merged commit b9520d6 into master Oct 22, 2026
23 checks passed
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