chore: editor namespace auto upgrade - #4142
Conversation
There was a problem hiding this comment.
💡 Harness Review
The relocation metadata covers the unconditional public editor API, but the new validation harness has a configuration-specific coverage gap and cannot be run reliably from paths containing spaces.
Reviewed commit dcf9a26
🤖 Helpful? 👍/👎
Removed obsolete custom property drawer for NetworkObject.OwnershipStatus.
…com/Unity-Technologies/com.unity.netcode.gameobjects into chore/editor-namespace-auto-upgrade
| @@ -0,0 +1,121 @@ | |||
| # API updater upgrade-path project | |||
There was a problem hiding this comment.
This README has too much info. The README from this project doesn't need info on how the upgrade was implemented, it just needs info on how it works.
If you want this project to be used also for future upgrades then this README shouldn't have any references to the 2.x -> 3.x upgrade specifically, otherwise future us might think we can pull this project out once that upgrade is released.
There was a problem hiding this comment.
This project and the tests associated with it will not be pertinent once v2.x.x becomes deprecated.
But I can remove the extra info...leaving it only provides the context as to why it is there in the first place.
There was a problem hiding this comment.
This should be much more straight forward now.
| { | ||
| "disableProjectUpdate": false, | ||
| "dependencies": { | ||
| "com.unity.netcode.gameobjects": "file:../../com.unity.netcode.gameobjects", |
There was a problem hiding this comment.
It might be nice to have N4E in here so we can be sure the upgrade doesn't introduce a conflict with N4E
There was a problem hiding this comment.
This was tested extensively locally. Nothing wrapped in UNIFIED_NETCODE is impacted by the namespace migration since anything that does reference NGO editor classes(etc) already has the change applied and no user has any script that could cause conflicts.
This validates without N4E an NGO users upgrading from v2.x to v3.x will automatically have everything that needs to be updated...updated.
Codecov ReportAll modified and coverable lines are covered by tests ✅ @@ Coverage Diff @@
## develop-3.x.x #4142 +/- ##
==============================================
Coverage 78.00% 78.00%
==============================================
Files 153 153
Lines 26260 26260
==============================================
Hits 20483 20483
Misses 5777 5777
Flags with carried forward coverage won't be shown. Click here to find out more.
|
|
/ci apiupdater |
|
/ci apiupdater |
|
/ci apiupdater |
Purpose of this PR
This PR handles the auto-upgrade namespace change from Unity.Netcode.Editor to Unity.Netcode.GameObjects.Editor.
It also includes a job that can be manually triggered to validate the auto-upgrade or the job itself could be added to nightly or the like.
Jira ticket
MTT-15570
Changelog
NA
Documentation
Testing & QA (How your changes can be verified during release Playtest)
Functional Testing
Manual testing :
Manual testing doneAutomated tests:
Covered by existing automated testsCovered by new automated testDoes the change require QA team to:
Review automated tests?Execute manual tests?Provide feedback about the PR?If any boxes above are checked the QA team will be automatically added as a PR reviewer.
Up-port
None
Backports
None