Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

CHANGE: Force reimport all assets into Unity 2022.3 LTS#2287

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
K-Tone merged 8 commits intodevelopfromanthony/reimport-into-2022.3
Nov 21, 2025

Conversation

@K-Tone
Copy link
Collaborator

@K-ToneK-Tone commentedNov 19, 2025
edited
Loading

Description

Here we force reimport all assets into Unity 2022.3 LTS with the means of this script:

using UnityEditor;using UnityEngine;using System.IO;public class ForceReserializeWindow : EditorWindow{    [MenuItem("Tools/Force Reserialize All Assets")]    public static void ShowWindow()    {        GetWindow<ForceReserializeWindow>("Force Reserialize Assets");    }    void OnGUI()    {        GUILayout.Label("Force all assets to be re-saved (reserialized).", EditorStyles.wordWrappedLabel);        if (GUILayout.Button("Force Reserialize All Assets"))        {            ForceReserializeAllAssets();        }    }    private static void ForceReserializeAllAssets()    {        // Get all asset paths in the project        string[] allAssetPaths = AssetDatabase.GetAllAssetPaths();                // Use ForceReserializeAssets with the option to reserialize both assets and metadata        // This loads, upgrades, and writes all data back to disk.        AssetDatabase.ForceReserializeAssets(allAssetPaths, ForceReserializeAssetsOptions.ReserializeAssetsAndMetadata);        Debug.Log($"Reserialization complete for {allAssetPaths.Length} assets.");    }}

This is done so that what we have on disk is actually 22.3 LTS data and not old data that is still usable due to compatibility paths. One of the benefits of this work is that with it landing, we won't get extra large diffs from doing seemingly small changes.

Testing status & QA

Local testing by dev.

Overall Product Risks

  • Complexity: Low
  • Halo Effect: Low

Checklist

Before review:

  • Changelog entry added.
    • Explains the change inChanged,Fixed,Added sections.
    • For API change contains an example snippet and/or migration example.
    • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
    • Jira port for the next release set as "Resolved".
  • Tests added/changed, if applicable.
    • Functional testsArea_CanDoX,Area_CanDoX_EvenIfYIsTheCase,Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.
  • Docs for new/changed API's.
    • Xmldoc cross references are set correctly.
    • Added explanation how the API works.
    • Usage code examples added.
    • The manual is updated, if needed.

During merge:

  • Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

@K-ToneK-Tone changed the titleAnthony/reimport into 2022.3CHANGE: Force reimport all assets into Unity 2022.3 LTSNov 20, 2025
@K-ToneK-Tone marked this pull request as ready for reviewNovember 20, 2025 10:08
@u-pr-agent
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

The PR contains a very small diff involving only configuration changes in `manifest.json` and one preset file, making it quick to review.
🏅 Score: 88

The changes appear consistent with a Unity version upgrade/maintenance, but the package updates in `manifest.json` are not explicitly documented in the PR description.
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Undocumented Package Updates

The PR updates several packages (including a major version bump forcom.unity.settings-manager) and addscom.unity.ai.navigation. These changes are not mentioned in the PR description, which focuses solely on asset reserialization. Please verify that these specific package versions are intended and compatible with the target Unity version.

"com.unity.ai.navigation":"1.1.6","com.unity.analytics":"3.8.1","com.unity.cinemachine":"2.10.3","com.unity.coding":"0.1.0-preview.24","com.unity.ide.rider":"3.0.36","com.unity.ide.visualstudio":"2.0.22","com.unity.settings-manager":"2.1.0","com.unity.test-framework":"1.4.5","com.unity.test-framework.build":"0.0.1-preview.12","com.unity.test-framework.performance":"3.0.3","com.unity.test-framework.utp-reporter":"1.1.0-preview","com.unity.textmeshpro":"3.0.7",
  • Update review

🤖 Helpful? Please react with 👍/👎 | Questions❓Please reach out in Slack#ask-u-pr-agent

@u-pr-agent
Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@codecov-github-com
Copy link

codecov-github-combot commentedNov 20, 2025
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

@@             Coverage Diff             @@##           develop    #2287      +/-   ##===========================================+ Coverage    76.81%   77.95%   +1.14%===========================================  Files          476      477       +1       Lines        88726    97418    +8692     ===========================================+ Hits         68155    75947    +7792- Misses       20571    21471     +900
FlagCoverage Δ
inputsystem_MacOS_2022.35.54% <ø> (+0.15%)⬆️
inputsystem_MacOS_2022.3_project75.49% <ø> (+0.79%)⬆️
inputsystem_MacOS_6000.05.32% <ø> (+0.14%)⬆️
inputsystem_MacOS_6000.0_project77.41% <ø> (+0.81%)⬆️
inputsystem_MacOS_6000.25.32% <ø> (+0.14%)⬆️
inputsystem_MacOS_6000.2_project77.41% <ø> (+0.81%)⬆️
inputsystem_MacOS_6000.35.32% <ø> (+0.14%)⬆️
inputsystem_MacOS_6000.3_project77.41% <ø> (+0.80%)⬆️
inputsystem_MacOS_6000.45.33% <ø> (+0.14%)⬆️
inputsystem_MacOS_6000.4_project77.41% <ø> (+0.80%)⬆️
inputsystem_MacOS_6000.55.33% <ø> (+0.14%)⬆️
inputsystem_MacOS_6000.5_project77.41% <ø> (+0.80%)⬆️
inputsystem_Ubuntu_2022.35.55% <ø> (+0.15%)⬆️
inputsystem_Ubuntu_2022.3_project75.28% <ø> (+0.79%)⬆️
inputsystem_Ubuntu_6000.05.33% <ø> (+0.14%)⬆️
inputsystem_Ubuntu_6000.0_project77.21% <ø> (+0.80%)⬆️
inputsystem_Ubuntu_6000.25.33% <ø> (+0.14%)⬆️
inputsystem_Ubuntu_6000.2_project77.21% <ø> (+0.80%)⬆️
inputsystem_Ubuntu_6000.35.33% <ø> (+0.14%)⬆️
inputsystem_Ubuntu_6000.3_project77.21% <ø> (+0.80%)⬆️
inputsystem_Ubuntu_6000.45.33% <ø> (+0.14%)⬆️
inputsystem_Ubuntu_6000.4_project77.22% <ø> (+0.80%)⬆️
inputsystem_Ubuntu_6000.55.33% <ø> (+0.14%)⬆️
inputsystem_Ubuntu_6000.5_project77.22% <ø> (+0.81%)⬆️
inputsystem_Windows_2022.35.54% <ø> (+0.15%)⬆️
inputsystem_Windows_2022.3_project75.62% <ø> (+0.79%)⬆️
inputsystem_Windows_6000.05.32% <ø> (+0.14%)⬆️
inputsystem_Windows_6000.0_project77.53% <ø> (+0.80%)⬆️
inputsystem_Windows_6000.25.32% <ø> (+0.14%)⬆️
inputsystem_Windows_6000.2_project77.53% <ø> (+0.80%)⬆️
inputsystem_Windows_6000.35.32% <ø> (+0.14%)⬆️
inputsystem_Windows_6000.3_project77.53% <ø> (+0.80%)⬆️
inputsystem_Windows_6000.45.33% <ø> (+0.14%)⬆️
inputsystem_Windows_6000.4_project77.54% <ø> (+0.80%)⬆️
inputsystem_Windows_6000.55.33% <ø> (+0.14%)⬆️
inputsystem_Windows_6000.5_project77.54% <ø> (+0.81%)⬆️

Flags with carried forward coverage won't be shown.Click here to find out more.

Files with missing linesCoverage Δ
Assets/Tools/ForceReserializeUtility.cs0.00% <ø> (ø)

... and66 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@josepmariapujol-unity
Copy link
Collaborator

Screenshot 2025-11-20 at 12 41 56

When I open the project I get this file created

@josepmariapujol-unity
Copy link
Collaborator

Should we commit it?

K-Tone reacted with thumbs up emoji

@K-ToneK-Tone requested review fromjosepmariapujol-unity and removed request forPauliusd01November 21, 2025 09:50
@K-Tone
Copy link
CollaboratorAuthor

CC@Pauliusd01 for viz, no functional changes here, just asset reimport

Copy link
Collaborator

@ekcohekcoh left a comment
edited
Loading

Choose a reason for hiding this comment

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

Great to get this upgrade, I skimmed through the files and changes look good. I was surprised to not see a similar script to the PR description check in though, why don't we add this to e.g. /Assets/Tools/AssetUtilties.cs (The menu option only - see no value in the window), so you only need to go to e.g. "QA Tools/Force Reserialize All Assets" next time we do this? It can also be handy to use when e.g. messing with C# generators or asset generation/format changes that requires you to "manually" upgrade all of these.

K-Tone reacted with thumbs up emoji
@ekcoh
Copy link
Collaborator

ekcoh commentedNov 21, 2025
edited
Loading

What I suggested is essentially this:

usingUnityEditor;usingUnityEngine;usingSystem.IO;staticclassAssetDatabaseMenuItems{// Use ForceReserializeAssets with the option to reserialize both assets and metadata// This loads, upgrades, and writes all data back to disk.[MenuItem("QA Tools/Force Reserialize All Assets")]staticvoidForceReserializeAllAssets()=>AssetDatabase.ForceReserializeAssets(AssetDatabase.GetAllAssetPaths(),ForceReserializeAssetsOptions.ReserializeAssetsAndMetadata);}```

@K-ToneK-Toneforce-pushed theanthony/reimport-into-2022.3 branch from9936753 to727c74fCompareNovember 21, 2025 10:24
Copy link
Collaborator

@josepmariapujol-unityjosepmariapujol-unity left a comment

Choose a reason for hiding this comment

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

Tested manually, approved

@K-ToneK-Tone merged commit83db18f intodevelopNov 21, 2025
127 checks passed
@K-ToneK-Tone deleted the anthony/reimport-into-2022.3 branchNovember 21, 2025 13:18
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ekcohekcohekcoh approved these changes

@josepmariapujol-unityjosepmariapujol-unityjosepmariapujol-unity approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@K-Tone@josepmariapujol-unity@ekcoh

[8]ページ先頭

©2009-2025 Movatter.jp