chore: Added upgrade-guide for 2.X --> 3.X migration - #4153
chore: Added upgrade-guide for 2.X --> 3.X migration#4153michalChrobot wants to merge 3 commits into
Conversation
|
we can land this after 3.X release |
|
Check the configuration's TOML syntax, permitted settings, and size limit. See the error below. Error message: Configuration content:# This is the configuration for the u-pr bot
# https://github.cds.internal.unity3d.com/unity/u-pr
# For configuration of this file:
# https://developer.portal.internal.unity.com/catalog/default/component/u-pr/docs/configuration/automatic_runs/
[github_app]
handle_pr_actions = ['opened', 'ready_for_review'] # PR events that auto-run pr_commands
pr_commands = ["/harness-review"] # what runs on those events
reviewer_commands = ["/harness-review"] # what runs when @u-pr is added as reviewer
|
|
/review |
There was a problem hiding this comment.
💡 Harness Review
The migration guide is generally aligned with the current API and behavior changes, but one documented replacement does not provide a valid migration for the normal legacy call pattern.
Reviewed commit 2393dcc
🤖 Helpful? 👍/👎
jabbacakes
left a comment
There was a problem hiding this comment.
Committed directly with changes that bring this upgrade guide in line with the Netcode for Entities one, as well as general style/wording/thoroughness updates.
Ready for review by @NoelStephensUnity and/or @EmandM
Purpose of this PR
Adding upgrade-guide in the same way as it was added for N4E. Feel free to make any corrections to it
Jira ticket
N/A
Documentation
Added migration-guide doc
Testing & QA (How your changes can be verified during release Playtest)
N/A
Up-port
N/A
Backports
N/A