From 80119e02986894e6f38d791309c1197d77329d75 Mon Sep 17 00:00:00 2001 From: Edward Ly Date: Fri, 21 Feb 2025 09:53:33 -0800 Subject: [PATCH 1/2] chore: adjust code owner files for app maintainers This change ensures that app maintainers are automatically requested for review whenever a pull request that changes code in their app is created. Signed-off-by: Edward Ly --- .github/CODEOWNERS | 62 ++++++++++++++++++++++------------------------ 1 file changed, 29 insertions(+), 33 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 0abdafc9fd824..60a0d25874c9c 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,37 +1,33 @@ # App maintainers -/apps/admin_audit/appinfo/info.xml @luka-nextcloud @blizzz -/apps/cloud_federation_api/appinfo/info.xml @mejo- -/apps/comments/appinfo/info.xml @edward-ly @Pytal -/apps/contactsinteraction/appinfo/info.xml @kesselb @SebastianKrupinski -/apps/contactsinteraction/lib @kesselb @SebastianKrupinski -/apps/contactsinteraction/tests @kesselb @SebastianKrupinski -/apps/dashboard/appinfo/info.xml @julien-nc @juliusknorr -/apps/dav/lib/CalDAV @st3iny @SebastianKrupinski @tcitworld -/apps/dav/lib/CardDAV @hamza221 @SebastianKrupinski -/apps/dav/tests/unit/CalDAV @st3iny @SebastianKrupinski @tcitworld -/apps/dav/tests/unit/CardDAV @hamza221 @SebastianKrupinski -/apps/encryption/appinfo/info.xml @come-nc @icewind1991 -/apps/federatedfilesharing/appinfo/info.xml @icewind1991 @danxuliu -/apps/federation/appinfo/info.xml @nfebe -/apps/files/appinfo/info.xml @skjnldsv @Pytal @ArtificialOwl @come-nc @artonge @icewind1991 @szaimen @susnux @nfebe -/apps/files_external/appinfo/info.xml @icewind1991 @artonge -/apps/files_reminders/appinfo/info.xml @Pytal -/apps/files_sharing/appinfo/info.xml @skjnldsv @come-nc -/apps/files_trashbin/appinfo/info.xml @Pytal @icewind1991 -/apps/files_versions/appinfo/info.xml @artonge @icewind1991 -/apps/oauth2/appinfo/info.xml @julien-nc @ChristophWurst -/apps/provisioning_api/appinfo/info.xml @provokateurin @nickvergessen -/apps/settings/appinfo/info.xml @Pytal @JuliaKirschenheuter -/apps/sharebymail/appinfo/info.xml @Altahrim -/apps/systemtags/appinfo/info.xml @Antreesy @marcelklehr -/apps/theming/appinfo/info.xml @skjnldsv @juliusknorr -/apps/twofactor_backupcodes/appinfo/info.xml @st3iny @miaulalala @ChristophWurst -/apps/updatenotification/appinfo/info.xml @Pytal @JuliaKirschenheuter -/apps/user_ldap/appinfo/info.xml @come-nc @blizzz -/apps/user_status/appinfo/info.xml @Antreesy @nickvergessen -/apps/weather_status/appinfo/info.xml @julien-nc @juliusknorr -/apps/webhook_listeners/appinfo/info.xml @come-nc @julien-nc -/apps/workflowengine/appinfo/info.xml @blizzz @juliusknorr +/apps/admin_audit/** @luka-nextcloud @blizzz +/apps/cloud_federation_api/** @mejo- +/apps/comments/** @edward-ly @Pytal +/apps/contactsinteraction/** @kesselb @SebastianKrupinski +/apps/dashboard/** @julien-nc @juliusknorr +/apps/dav/**/CalDAV/** @st3iny @SebastianKrupinski @tcitworld +/apps/dav/**/CardDAV/** @hamza221 @SebastianKrupinski +/apps/encryption/** @come-nc @icewind1991 +/apps/federatedfilesharing/** @icewind1991 @danxuliu +/apps/federation/** @nfebe +/apps/files/** @skjnldsv @Pytal @ArtificialOwl @come-nc @artonge @icewind1991 @szaimen @susnux @nfebe +/apps/files_external/** @icewind1991 @artonge +/apps/files_reminders/** @Pytal +/apps/files_sharing/** @skjnldsv @come-nc +/apps/files_trashbin/** @Pytal @icewind1991 +/apps/files_versions/** @artonge @icewind1991 +/apps/oauth2/** @julien-nc @ChristophWurst +/apps/provisioning_api/** @provokateurin @nickvergessen +/apps/settings/** @Pytal @JuliaKirschenheuter +/apps/sharebymail/** @Altahrim +/apps/systemtags/** @Antreesy @marcelklehr +/apps/theming/** @skjnldsv @juliusknorr +/apps/twofactor_backupcodes/** @st3iny @miaulalala @ChristophWurst +/apps/updatenotification/** @Pytal @JuliaKirschenheuter +/apps/user_ldap/** @come-nc @blizzz +/apps/user_status/** @Antreesy @nickvergessen +/apps/weather_status/** @julien-nc @juliusknorr +/apps/webhook_listeners/** @come-nc @julien-nc +/apps/workflowengine/** @blizzz @juliusknorr # Frontend expertise /apps/files/src* @skjnldsv From 95dd9194b77645878379474711466ae1039697e6 Mon Sep 17 00:00:00 2001 From: Edward Ly Date: Mon, 24 Feb 2025 09:08:14 -0800 Subject: [PATCH 2/2] fix: remove duplicate code owners declaration Signed-off-by: Edward Ly --- .github/CODEOWNERS | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 60a0d25874c9c..e983cf61b7a9e 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -21,7 +21,6 @@ /apps/sharebymail/** @Altahrim /apps/systemtags/** @Antreesy @marcelklehr /apps/theming/** @skjnldsv @juliusknorr -/apps/twofactor_backupcodes/** @st3iny @miaulalala @ChristophWurst /apps/updatenotification/** @Pytal @JuliaKirschenheuter /apps/user_ldap/** @come-nc @blizzz /apps/user_status/** @Antreesy @nickvergessen