PR review fixes

ScriptableRenderPipeline/HDRenderPipeline/Sky/SkyManager.cs

             {
                 CoreUtils.SetRenderTarget(cmd, dest, ClearFlag.None, 0, (CubemapFace)i);
                 propertyBlock.SetTexture("_MainTex", source);
-                propertyBlock.SetFloat("_faceIndex", i);
+                propertyBlock.SetFloat("_faceIndex", (float)i);
                 cmd.DrawProcedural(Matrix4x4.identity, m_BlitCubemapMaterial, 0, MeshTopology.Triangles, 3, 1, propertyBlock);
             }
 

ScriptableRenderPipeline/HDRenderPipeline/Sky/SkySettings.cs

                 hash = hash * 23 + rotation.GetHashCode();
                 hash = hash * 23 + exposure.GetHashCode();
                 hash = hash * 23 + multiplier.GetHashCode();
-                hash = hash * 23 + ((int)resolution).GetHashCode(); // Enum.GetHashCode generates garbade on .NET 3.5... Works fine on 4.6+. Wtf !?
+
+                // TODO: Fixme once we switch to .Net 4.6+
+                //>>>
+                hash = hash * 23 + ((int)resolution).GetHashCode(); // Enum.GetHashCode generates garbade on .NET 3.5... Wtf !?
+                //<<<
+
                 hash = hash * 23 + updatePeriod.GetHashCode();
                 hash = lightingOverride != null ? hash * 23 + rotation.GetHashCode() : hash;
                 return hash;