Restructure repo (#7355)

* Restructure repo

* Update .github paths

* Update repoconfig

* Fix repoconfig

* Fix2 for repoconfig (hybrid)

* Fix3

.gitattributes

@@ -52,4 +52,4 @@ LightingData.asset binary
 *.[gG][zZ] filter=lfs diff=lfs merge=lfs -text
 *.[rR][aA][rR] filter=lfs diff=lfs merge=lfs -text
 *.[tT][aA][rR] filter=lfs diff=lfs merge=lfs -text
-TestProjects/ShaderGraph/Assets/TestbedAssets/Textures/Texture2DArray/*.asset filter=lfs diff=lfs merge=lfs -text
+**/ShaderGraph/Assets/TestbedAssets/Textures/Texture2DArray/*.asset filter=lfs diff=lfs merge=lfs -text

.github/CODEOWNERS

@@ -1,20 +1,18 @@
 # Code Ownership File
 
 # Universal RP General
-/com.unity.render-pipelines.universal/ @Unity-Technologies/universal-rp-team @Unity-Technologies/gfx-qa-urp
+/Packages/com.unity.render-pipelines.universal/ @Unity-Technologies/universal-rp-team @Unity-Technologies/gfx-qa-urp
 
 # Universal RP 2D Renderer
-/com.unity.render-pipelines.universal/Editor/2D @Unity-Technologies/2d-graphics
-/com.unity.render-pipelines.universal/Runtime/2D @Unity-Technologies/2d-graphics
-/com.unity.render-pipelines.universal/Shaders/2D @Unity-Technologies/2d-graphics
-/com.unity.render-pipelines.universal/Editor/ShaderGraph/MasterNodes @Unity-Technologies/2d-graphics
+/Packages/com.unity.render-pipelines.universal/Editor/2D @Unity-Technologies/2d-graphics
+/Packages/com.unity.render-pipelines.universal/Runtime/2D @Unity-Technologies/2d-graphics
+/Packages/com.unity.render-pipelines.universal/Shaders/2D @Unity-Technologies/2d-graphics
+/Packages/com.unity.render-pipelines.universal/Editor/ShaderGraph/MasterNodes @Unity-Technologies/2d-graphics
 
 # Shader Graph
-/com.unity.shadergraph/ @Unity-Technologies/shader-graph
+/Packages/com.unity.shadergraph/ @Unity-Technologies/shader-graph
 
 # Test systems
 /.github/ @Unity-Technologies/gfx-sdets
 /.yamato/ @Unity-Technologies/gfx-sdets
-/katana-ci/ @Unity-Technologies/gfx-sdets
 /Tools/ @Unity-Technologies/gfx-sdets
-/com.unity.testframework.graphics/ @Unity-Technologies/gfx-sdets

.github/pr-commenter.yml

@@ -10,30 +10,30 @@ comment:
   snippets:
     - id: HDRP
       files:
-        - com.unity.render-pipelines.high-definition-config/**/*
-        - com.unity.render-pipelines.high-definition/**/*
-        - com.unity.testing.hdrp/**/*
-        - TestProjects/HDRP*/**/*
+        - Packages/com.unity.render-pipelines.high-definition-config/**/*
+        - Packages/com.unity.render-pipelines.high-definition/**/*
+        - Tests/SRPTests/Packages/com.unity.testing.hdrp/**/*
+        - Tests/SRPTests/Projects/HDRP*/**/*
       body: |
         **HDRP**
         `/jobDefinition/.yamato%2Fall-hdrp.yml%23PR_HDRP_2021.3`
         With changes to HDRP packages, you should also run
         `/jobDefinition/.yamato%2Fall-lightmapping.yml%23PR_Lightmapping_2021.3`
     - id: URP
       files:
-        - com.unity.render-pipelines.universal/**/*
-        - com.unity.testing.urp/**/*
-        - TestProjects/Universal*/**/*
+        - Packages/com.unity.render-pipelines.universal/**/*
+        - Tests/SRPTests/Packages/com.unity.testing.urp/**/*
+        - Tests/SRPTests/Projects/Universal*/**/*
       body: |
         **URP**
         `/jobDefinition/.yamato%2Fall-urp.yml%23PR_URP_2021.3`
         With changes to URP packages, you should also run
         `/jobDefinition/.yamato%2Fall-lightmapping.yml%23PR_Lightmapping_2021.3`
     - id: ShaderGraph
       files:
-        - com.unity.shadergraph/**/*
-        - TestProjects/ShaderGraph*/**/*
-        - com.unity.render-pipelines.universal/Editor/ShaderGraph/*
+        - Packages/com.unity.shadergraph/**/*
+        - Tests/SRPTests/Projects/ShaderGraph*/**/*
+        - Packages/com.unity.render-pipelines.universal/Editor/ShaderGraph/*
       body: |
         **Shader Graph**
         `/jobDefinition/.yamato%252Fall-shadergraph.yml%2523PR_ShaderGraph_2021.3`
@@ -42,15 +42,15 @@ comment:
          /jobDefinition/.yamato%2Fall-shadergraph_builtin_lighting.yml%23PR_ShaderGraph_BuiltIn_Lighting_2021.3
     - id: VFX
       files:
-        - com.unity.visualeffectgraph/**/*
-        - com.unity.testing.visualeffectgraph/**/*
-        - TestProjects/VisualEffectGraph*/**/*
+        - Packages/com.unity.visualeffectgraph/**/*
+        - Tests/SRPTests/Packages/com.unity.testing.visualeffectgraph/**/*
+        - Tests/SRPTests/Projects/VisualEffectGraph*/**/*
       body: |
         **VFX**
         `/jobDefinition/.yamato%252Fall-vfx.yml%2523PR_VFX_2021.3`
     - id: Core
       files:
-        - com.unity.render-pipelines.core/**/*
+        - Packages/com.unity.render-pipelines.core/**/*
       body: |
         **SRP Core**
         You could run ABV on your branch before merging your PR, but it will start A LOT of jobs. Please be responsible about it and run it only when you feel the PR is ready:

.repoconfig

@@ -19,19 +19,18 @@ path-ignore=<<EOT
  Tools
  |com\.unity\.template-hd                                   # Todo: Exclude the Library folder of the checks only (keep checks for C# code files)
  |com\.unity\.template-universal
- |LocalTestProjects
- |TestProjects\/.*\/Library
- |TestProjects\/HDRP_DXR_Tests                              # Ideally all test projects would be included, but doing it wone by one is less overwhelming for teams
- |TestProjects\/HDRP_HybridTests                            # So remove the test projects one by one as they are progressively being reformatted
- |TestProjects\/HDRP_PerformanceTests                       # New test projects folder will automatically be included in the formatting checks since they aren't in that list
- |TestProjects\/HDRP_RuntimeTests
- |TestProjects\/Shadergraph
- |TestProjects\/ShadergraphUniversalStereo
- |TestProjects\/SRP_SmokeTest
- |TestProjects\/UniversalGfxTestStereo
- |TestProjects\/UniversalHybridTest
- |TestProjects\/VisualEffectGraph_HDRP
- |TestProjects\/VisualEffectGraph_URP
+ |Tests\/SRPTests\/Projects\/.*\/Library
+ |Tests\/SRPTests\/Projects\/HDRP_HybridTests                            # Remove the test projects one by one as they are progressively being reformatted
+ |Tests\/SRPTests\/Projects\/UniversalHybridTest                         # Remove the test projects one by one as they are progressively being reformatted
+ |Tests\/SRPTests\/Projects\/HDRP_DXR_Tests                              # Ideally all test projects would be included, but doing it wone by one is less overwhelming for teams
+ |Tests\/SRPTests\/Projects\/HDRP_PerformanceTests                       # New test projects folder will automatically be included in the formatting checks since they aren't in that list
+ |Tests\/SRPTests\/Projects\/HDRP_RuntimeTests
+ |Tests\/SRPTests\/Projects\/Shadergraph
+ |Tests\/SRPTests\/Projects\/ShadergraphUniversalStereo
+ |Tests\/SRPTests\/Projects\/SRP_SmokeTest
+ |Tests\/SRPTests\/Projects\/UniversalGfxTestStereo
+ |Tests\/SRPTests\/Projects\/VisualEffectGraph_HDRP
+ |Tests\/SRPTests\/Projects\/VisualEffectGraph_URP
 )/|
 # rooted file paths
 ^(
@@ -41,11 +40,14 @@ EOT
 [verify]
 path=<<EOT
 # general rule
-^[:a-z0-9./_-]+$                                            # technically ':' not allowed, but let the global rule in format.ini do the more thorough test for c: style abs paths
+^[:a-z0-9./_-]+$                                                          # technically ':' not allowed, but let the global rule in format.ini do the more thorough test for c: style abs paths
 # exceptions
-|^com\.unity\.[^\/]*\/\w+~\/[^&]+$                          # allows editor excluded folders, like Documentation~, for core package
-|^com\.unity\.[^\/]*\/Packages\/com\.unity\.[^\/]*\/\w+~\/  # as above but for embedded packages
-|^com\.unity\.[^\/]*\/[^@]+@\d+x\.                          # industry standard for icons with differnet resolution
-|^TestProjects\/                                            # allow all names for test files
-|^com\.unity\.testing\.hdrp                                 # as above but for testing packages
+|^com\.unity\.[^\/]*\/\w+~\/[^&]+$                                        # allows editor excluded folders, like Documentation~, for core package
+|^com\.unity\.[^\/]*\/Packages\/com\.unity\.[^\/]*\/\w+~\/                # as above but for embedded packages
+|^com\.unity\.[^\/]*\/[^@]+@\d+x\.                                        # industry standard for icons with different resolution
+|^(Packages)\/com\.unity\.[^\/]*\/\w+~\/	                                # allows editor excluded folders, like Documentation~, for core package
+|^(Packages)\/com\.unity\.[^\/]*\/Packages\/com\.unity\.[^\/]*\/\w+~\/    # as above but for embedded packages
+|^(Packages)\/com\.unity\.[^\/]*\/Third\sParty\sNotices\..*               # allows Spaces in names for Third Party Notices.md
+|^(Packages)\/com\.unity\.[^\/]*\/[^@]+@\d+x\.                            # industry standard for icons with different resolutions
+|^Tests\/SRPTests\/                                                       # allow all names for test files
 EOT