From 77d199910b0b0ce74d9a943ddc21eb948b2f8eff Mon Sep 17 00:00:00 2001 From: Gusted Date: Fri, 15 Sep 2023 20:41:08 +0200 Subject: [PATCH] [META] Add CODEOWNERS files - As per https://codeberg.org/forgejo/discussions/issues/53#issuecomment-1070207 - Using the `CODEOWNERS` feature it should speed up the development process for contributors as for the most common PRs the right reviewers will be added by Forgejo automatically. They can be added very precisely according to the changed files of the PR. - This feature is implemented in v1.21, which means it's not available on Codeberg. (cherry picked from commit 1511ef1c803f31fcf2887611982a114465f3be60) (cherry picked from commit 99999e3a034b719dcefc63951538d917dfd9e496) (cherry picked from commit 0b0dd6f7a95ce718c8ff73e8e6fe1b660a4e3277) (cherry picked from commit d42940034f7fe1f1f706e4fff8f9b5c8465003fc) (cherry picked from commit 5be6e7d2542e6915d1df3d9f29a8504f851b8f52) (cherry picked from commit bf5b3994217b1de4470f7bd6201543e9fc688675) (cherry picked from commit 52c9f9fd8af54ed95361e3ee79fd8001a5461135) (cherry picked from commit 7e03d2829ba683e0b865773c7fbb2a1f78f0cfdb) --- CODEOWNERS | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) create mode 100644 CODEOWNERS diff --git a/CODEOWNERS b/CODEOWNERS new file mode 100644 index 0000000000..10e9d52b35 --- /dev/null +++ b/CODEOWNERS @@ -0,0 +1,32 @@ +# This file describes the expected reviewers for a PR based on the changed +# files. Unlike what the name of the file suggests they don't own the code, but +# merely have a good understanding of that area of the codebase and therefore +# are usually suited as a reviewer. + + +# Please mind the alphabetic order of reviewers. + +# Files related to the CI of the Forgejo project. +.forgejo/.* @dachary @earl-warren + +# Files related to frontend development. + +# Javascript and CSS code. +web_src/.* @caesar @crystal @gusted + +# HTML templates used by the backend. +templates/.* @caesar @crystal @gusted + +# Files related to Go development. + +# The modules usually don't require much knowledge about Forgejo and could +# be reviewed by Go developers. +modules/.* @dachary @earl-warren @gusted + +# Models has code related to SQL queries, general database knowledge and XORM. +models/.* @dachary @earl-warren @gusted + +# The routers directory contains the most amount code that requires a good grasp +# of how Forgejo comes together. It's tedious to write good integration testing +# for code that lives in here. +routers/.* @dachary @earl-warren @gusted