Apply suggestions from code review

Co-authored-by: Emma <emma.mcmillan@unity3d.com>

Author: NoelStephensUnity

.yamato/_triggers.yml

@@ -128,10 +128,8 @@ pr_code_changes_checks:
 
 
 # Unified (NGO + N4E) validation, on demand.
-# The unified tests run automatically as part of pr_code_changes_checks and develop_nightly (same as
-# the CMB service tests do). This job only exists so they can also be kicked off by commenting
-# "/ci unified", for PRs that pr_code_changes_checks does not cover (drafts, doc-only changes, or
-# branches targeting something other than develop / develop-3.x.x / release).
+# This job allows the Unified tests to be kicked off by commenting "/ci unified". 
+# This is useful for PRs where pr_code_changes_checks doesn't trigger.
 unified_pr_checks:
   name: Unified (NGO + N4E) checks [on demand]
   dependencies:

com.unity.netcode.gameobjects/Runtime/Transports/Unified/UnifiedNetcodeTransport.cs

@@ -124,13 +124,6 @@ public void OnUpdate(ref SystemState state)
     [UpdateBefore(typeof(RpcSystem))]
     internal partial class UnifiedNetcodeUpdateSystem : SystemBase
     {
-        // Note: this must be the parameterless ComponentSystemBase.OnCreate override, not the
-        // ISystem-style OnCreate(ref SystemState). With the ISystem signature this is just a new
-        // method that Entities never calls, so neither RequireForUpdate takes effect and OnUpdate
-        // runs from the very first world tick - which is before StartClient/StartServer have
-        // assigned Transport and NetworkManager below, and therefore a NullReferenceException.
-        // CreateSingleWorldHost calls AppendWorldToCurrentPlayerLoop, so the world can tick in the
-        // window between world creation and the transport being started.
         protected override void OnCreate()
         {
             RequireForUpdate<RpcCollection>();
@@ -163,9 +156,6 @@ public void SendRpc(TransportRpc rpc, Entity connectionEntity)
 
         protected override void OnUpdate()
         {
-            // Belt and braces alongside the RequireForUpdate gating in OnCreate: these are only
-            // assigned once the transport is started but the world can already be in the player loop
-            // before that happens. Exit early under this scenario.
             if (NetworkManager == null || Transport == null)
             {
                 return;