Skip to content
Open
119 changes: 55 additions & 64 deletions .github/CODEOWNERS
Comment thread
AGindinson marked this conversation as resolved.
Original file line number Diff line number Diff line change
Expand Up @@ -8,87 +8,78 @@
# @ghost is used to make a pattern have no owners. It is a sentinel GitHub user
# that takes the place of deleted users.

# No global owners because we don't really want e.g. changing the root
# CMakeLists.txt file to always ping a bunch of people.
# Default to members of the IREE TSC
* @AGindinson @devtbi @jschuhmacher @schuermans-roofline
Comment on lines +11 to +12

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You could also create a github team for the TSC (https://github.com/orgs/iree-org/teams)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good suggestion, thanks Scott! I'll probably do this as a follow-up, we might as well revise/set up teams for component reviews


# Third-Party Code
/.gitmodules @ScottTodd @stellaraccident
/third_party/ @ScottTodd @stellaraccident
/.gitmodules @AGindinson @jschuhmacher @schuermans-roofline
/third_party/ @AGindinson @jschuhmacher @schuermans-roofline
# Except for routinely-updated submodules
/third_party/llvm-project @ghost
/third_party/llvm-project.branch-pin @ghost
/third_party/stablehlo @ghost
/third_party/torch-mlir @ghost
/third_party/llvm-project @AGindinson @schuermans-roofline
/third_party/stablehlo @jschuhmacher @maxbartel
/third_party/torch-mlir @AGindinson @maxbartel @ziereis

# Bindings
/runtime/bindings/python/ @stellaraccident
/runtime/bindings/tflite/ @benvanik

# Integrations
/integrations/ @benvanik @stellaraccident
/integrations/tensorflow/ @stellaraccident
/integrations/tensorflow/test/**/iree_tfl_tests/ @rsuderman
/runtime/bindings/python/ @devtbi @Manewing @schuermans-roofline @stellaraccident
/runtime/bindings/tflite/ @AGindinson @benvanik @maxbartel

# Experimental
# It's experimental, but we still don't want any old directory added here.
/experimental/ @benvanik @stellaraccident
/experimental/ @benvanik @jschuhmacher @maxbartel @stellaraccident

# Infra Top-Level Directories
/build_tools/ @Groverkss @kuhar
/build_tools/python_deploy/ @stellaraccident
/build_tools/scripts/ @Groverkss @kuhar
/build_tools/third_party/ @ScottTodd @stellaraccident
/.github/ @amd-eochoalo @kuhar @Groverkss

# llvm-external-projects
/llvm-external-projects/ @stellaraccident
/.github/ @AGindinson @kuhar @schuermans-roofline
/build_tools/ @AGindinson @jschuhmacher @schuermans-roofline
/build_tools/python_deploy/ @chrsmcgrr @maxbartel @stellaraccident
/build_tools/scripts/ @AGindinson @Manewing @schuermans-roofline
/build_tools/third_party/ @AGindinson @devtbi @stellaraccident

# Other Top-Level Directories
/docs/ @ScottTodd
/tools/ @benvanik
/docs/ @AGindinson @devtbi @jschuhmacher @schuermans-roofline
/tools/ @benvanik @devtbi @schuermans-roofline

# Tests
/tests/external/iree-test-suites @Groverkss @MaheshRavishankar @kuhar
/tests/external/iree-test-suites @AGindinson @jschuhmacher @roberto-laudani @ziereis

# Compiler
/compiler/src/iree/compiler/ @benvanik
/compiler/src/iree/compiler/API/ @benvanik @IanWood1
/compiler/src/iree/compiler/Codegen/ @MaheshRavishankar
/compiler/src/iree/compiler/Codegen/Common @hanhanW @Max191 @MaheshRavishankar @qedawkins
/compiler/src/iree/compiler/Codegen/**/*Vector* @amd-eochoalo @Groverkss @hanhanW @sommerlukas
/compiler/src/iree/compiler/Codegen/Common/GPU @qedawkins @Groverkss @kuhar
/compiler/src/iree/compiler/Codegen/Dialect/Codegen @Max191 @MaheshRavishankar @qedawkins
/compiler/src/iree/compiler/Codegen/Dialect/CPU @hanhanW
/compiler/src/iree/compiler/Codegen/Dialect/GPU @qedawkins @Max191 @Groverkss @nirvedhmeshram @krzysz00
/compiler/src/iree/compiler/Codegen/Dialect/PCF @qedawkins
/compiler/src/iree/compiler/Codegen/Dialect/VectorExt @Groverkss
/compiler/src/iree/compiler/Codegen/ExternalInterfaces @hanhanW @Max191 @jtuyls
/compiler/src/iree/compiler/Codegen/LLVMCPU/ @hanhanW
/compiler/src/iree/compiler/Codegen/LLVMGPU/ @qedawkins @kuhar @Groverkss @nirvedhmeshram @krzysz00 @Max191
/compiler/src/iree/compiler/Codegen/SPIRV/ @kuhar
/compiler/src/iree/compiler/ConstEval/ @hanhanW @MaheshRavishankar
/compiler/src/iree/compiler/Dialect/Encoding/ @hanhanW @Max191 @jtuyls
/compiler/src/iree/compiler/Dialect/Flow/ @hanhanW @MaheshRavishankar @IanWood1
/compiler/src/iree/compiler/Dialect/LinalgExt/ @hanhanW @MaheshRavishankar @Groverkss @Max191 @IanWood1 @bangtianliu
/compiler/src/iree/compiler/Dialect/Stream/ @benvanik @hanhanW @MaheshRavishankar
/compiler/src/iree/compiler/Dialect/TensorExt/ @hanhanW @MaheshRavishankar @IanWood1
/compiler/src/iree/compiler/Dialect/Vulkan/ @kuhar
/compiler/src/iree/compiler/DispatchCreation/ @MaheshRavishankar @IanWood1
/compiler/src/iree/compiler/GlobalOptimization/ @MaheshRavishankar @IanWood1
/compiler/src/iree/compiler/InputConversion/ @MaheshRavishankar
/compiler/src/iree/compiler/Pipelines/ @benvanik @IanWood1
/compiler/src/iree/compiler/Preprocessing/ @qedawkins @MaheshRavishankar
/compiler/plugins/ @jschuhmacher @Manewing @schuermans-roofline
/compiler/plugins/input @AGindinson @jschuhmacher @maxbartel
/compiler/plugins/target @devtbi @egebeysel @jschuhmacher @Manewing
/compiler/src/ @AGindinson @benvanik @jschuhmacher
/compiler/src/iree/compiler/API/ @benvanik @jschuhmacher @maxbartel @schuermans-roofline
/compiler/src/iree/compiler/Codegen/ @AGindinson @egebeysel @phemashekar
/compiler/src/iree/compiler/Codegen/Common @AGindinson @egebeysel @phemashekar
/compiler/src/iree/compiler/Codegen/**/*Vector* @Groverkss @hanhanW @sommerlukas
/compiler/src/iree/compiler/Codegen/Common/GPU @AGindinson @egebeysel @ziereis
/compiler/src/iree/compiler/Codegen/Dialect/CPU @AGindinson @egebeysel @Manewing @hanhanW
/compiler/src/iree/compiler/Codegen/Dialect/GPU @AGindinson @egebeysel @Groverkss @krzysz00
/compiler/src/iree/compiler/Codegen/LLVMCPU/ @AGindinson @egebeysel @Manewing @phemashekar @ziereis
/compiler/src/iree/compiler/Codegen/LLVMGPU/ @AGindinson @egebeysel @Groverkss @krzysz00 @ziereis
/compiler/src/iree/compiler/Codegen/SPIRV/ @AGindinson @devtbi @egebeysel
/compiler/src/iree/compiler/ConstEval/ @AGindinson @jschuhmacher @MaheshRavishankar
/compiler/src/iree/compiler/Dialect/Encoding/ @egebeysel @hanhanW @jschuhmacher @phemashekar
/compiler/src/iree/compiler/Dialect/Flow/ @devtbi @jschuhmacher @Manewing @mbknust @ziereis
/compiler/src/iree/compiler/Dialect/LinalgExt/ @AGindinson @egebeysel @jschuhmacher @ziereis
/compiler/src/iree/compiler/Dialect/Stream/ @benvanik @devtbi @Manewing @ziereis
/compiler/src/iree/compiler/Dialect/TensorExt/ @devtbi @egebeysel @ziereis
/compiler/src/iree/compiler/Dialect/Vulkan/ @AGindinson @devtbi
/compiler/src/iree/compiler/DispatchCreation/ @AGindinson @egebeysel @phemashekar @ziereis
/compiler/src/iree/compiler/GlobalOptimization/ @AGindinson @egebeysel @Manewing @phemashekar @ziereis
/compiler/src/iree/compiler/InputConversion/ @AGindinson @jschuhmacher @Manewing
/compiler/src/iree/compiler/Pipelines/ @benvanik @jschuhmacher @Manewing
/compiler/src/iree/compiler/Preprocessing/ @AGindinson @egebeysel @Manewing @phemashekar @ziereis

# Compiler Plugins
/compiler/plugins/input/StableHLO/ @MaheshRavishankar @rsuderman
/compiler/plugins/input/Torch/ @keshavvinayak01 @rsuderman @zjgarvey
/compiler/plugins/input/TOSA/ @MaheshRavishankar @rsuderman
/compiler/plugins/target/ROCM/ @kuhar @krzysz00 @jtuyls @Yu-Zhewen
/compiler/plugins/input/StableHLO/ @jschuhmacher @maxbartel
/compiler/plugins/input/Torch/ @AGindinson @rsuderman @ziereis @zjgarvey
/compiler/plugins/input/TOSA/ @AGindinson @chrsmcgrr @jschuhmacher @Manewing
/compiler/plugins/target/ROCM/ @jtuyls @krzysz00 @kuhar @Yu-Zhewen

# Runtime
/runtime/src/iree/ @benvanik
/runtime/src/iree/hal/drivers/amdgpu/ @benvanik @AWoloszyn
/runtime/src/iree/hal/drivers/cuda/ @antiagainst
/runtime/src/iree/hal/drivers/hip/ @AWoloszyn
/runtime/src/iree/hal/drivers/metal/ @antiagainst
/runtime/src/iree/hal/drivers/vulkan/ @antiagainst
/runtime/src/ @benvanik @devtbi @Manewing @schuermans-roofline
/runtime/src/iree/builtins/ukernel @egebeysel @jschuhmacher @phemashekar
/runtime/src/iree/hal/drivers/amdgpu/ @benvanik @krzysz00 @kuhar
/runtime/src/iree/hal/drivers/cuda/ @antiagainst @devtbi
/runtime/src/iree/hal/drivers/hip/ @AWoloszyn @krzysz00
/runtime/src/iree/hal/drivers/metal/ @antiagainst @devtbi
/runtime/src/iree/hal/drivers/vulkan/ @AGindinson @benvanik @devtbi
25 changes: 19 additions & 6 deletions docs/website/docs/developers/general/contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -151,17 +151,30 @@ track of those who have made significant contributions to the project.
* The source control history remains the most accurate source for individual
contributions.

The
[`.github/CODEOWNERS` file](https://github.com/iree-org/iree/blob/main/.github/CODEOWNERS)
lets maintainers opt in to PR reviews modifying certain paths.

* Review is not required from a code owner, though it is recommended.

The
[`MAINTAINERS.md` file](https://github.com/iree-org/iree/blob/main/MAINTAINERS.md)
documents the Technical Steering Committee (TSC) and official maintainers for
project components.

The
[`.github/CODEOWNERS` file](https://github.com/iree-org/iree/blob/main/.github/CODEOWNERS)
designates reviewers for specific components of the project. Maintainers of the
project are assigned as reviewers by default, so that they serve as the first
point of contact and help find a reviewer in case they can't perform the review
themselves.

A strong, recent history of quality commits to a certain part of the project is
generally enough to nominate oneself as a CODEOWNER - subject to maintainer
approval, such participation in the project would be very welcome.

An approval on the PR generally doesn't have to come from a designated code
owner. A single approval by a project member with write access is technically
sufficient to merge the change, however authors of larger PRs that affect
multiple components should allow more time for several code owners and other
project members to review. In such cases, preceding the change with a RFC and
aligning with code owners/maintainers on the design is also a recommended
practice - this allows to minimize time spent in the final PR review stage.

## :octicons-code-16: Coding policies

### :octicons-pencil-16: Coding style guidelines
Expand Down
Loading