Skip to content

Conversation

phi-lira
Copy link
Contributor


Purpose of this PR

This PR increases coverage of code ownership for URP team.


Testing status

Not needed.


Comments to reviewers

Universal RP team ownership means code review
URP Trunk Guardian ownership means review or URP jobs are green before approving for merge.

@phi-lira phi-lira requested a review from a team as a code owner October 18, 2021 12:10
@github-actions github-actions bot added the sdet label Oct 18, 2021
@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page).
See the PR template for more information.
Thank you!

# Universal RP General
# Universal RP Team and URP Guardian
/com.unity.render-pipelines.universal/ @Unity-Technologies/universal-rp-team @Unity-Technologies/urp-trunk-guardian
/com.unity.testing.urp-upgrade/ @Unity-Technologies/universal-rp-team @Unity-Technologies/url-trunk-guardian
Copy link
Contributor

Choose a reason for hiding this comment

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

typo in this line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch. Fixed.

@phi-lira phi-lira merged commit cc5837c into master Oct 19, 2021
@phi-lira phi-lira deleted the universal/update-code-owner branch October 19, 2021 07:01
odbb added a commit that referenced this pull request Oct 19, 2021
* master:
  Enable iris normal for Eye shader (#5880)
  Update to HDRP Asset analytics (#6060)
  [HDPR] Update reference screenshots for Linux Vulkan
  Update 206_Motion_Vectors.png (#6046)
  Updated code owner for URP (#6052)
  [URP] Re-enable ios pbr tests #5983
  Fix error when using gizmos with camera stack in editor #5913
  emitting UI geometry for offscreen cameras too (case 1344969 fix) #5894
  Remove deprecated UNITY_USE_NATIVE_HDR keyword from shader code. #5569
  [XR][URP] Fix issue with vignette location in XR #5471 #5471
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants