From d6d6b8c5cecf083eadb7fca6ac167958a53dda72 Mon Sep 17 00:00:00 2001 From: Pierre Siddall <43399998+Pierre-siddall@users.noreply.github.com> Date: Fri, 28 Aug 2026 15:58:47 +0100 Subject: [PATCH 1/4] Add Erica and I to CODEOWNERS file --- .github/CODEOWNERS | 80 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 80 insertions(+) create mode 100644 .github/CODEOWNERS diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 00000000..b4c9f298 --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,80 @@ += Code Owners = + +The purpose of this page is to list ownership of the various components +held within the MOCI repository. + +Please consider the following: + * Ownership is not a commitment for the individual to undertake all work + requested or all reviewing for the owned section + * Any planned changes to a section should be discussed with the code owner + to ensure plans are appropriate + * Code owners should be given the option to be a science/technical reviewer + for anyu code that alters their section + * Any interface between components needss to be agreed by all affected parties + +Contact details for code owners: https://code.metoffice.gov.uk/trac/home/wiki/UserList + += Coupling Applications: Contacts = +{{{ +Area Owner +UM umsysteam@metoffice.gov.uk +NEMO timgraham +SI3 davestorkey +CICE davestorkey +MEDUSA NOC_southampton +LFRic timgraham richardhill +Wave juancastillo +Rivers dancopsey juancastillo +OASIS_MCT harryshepherd +XIOS harryshepherd +CPMIP +mule umsysteam@metoffice.gov.uk +iris ml-avd-support@metoffice.gov.uk +}}} + += MOCI Components = +== Applications == +{{{ +Area Owner +rose-stem/ @ericaneininger @Pierre-siddall +Coupled_Drivers @Pierre-siddall @ericaneininger +Postprocessing @ericaneininger +Fortran_Cap +Coupling_Documentation dancopsey +}}} + + +== Configurations == +{{{ +Area Owner +fcm-make +Infrastucture_Builds +GC4 +GC5 +Global_Coupled_NWP +UKESM janemulcahy +}}} + +== Utilities == +{{{ +Area Owner +moci_lib @ericaneininger +moci_testing_scripts +dashboard @ericaneininger +ozone_redistribution @ericaneininger +mean_nemo +verification_scripts +NGMS_utils +}}} + +== Rose meta data == +{{{ +Area Owner +archive_and_meaning @ericaneininger +coupled_drivers +ocean_ice davestorkey +fcm_make_ocean richardhill +GA7 +GC3 +GC3.1 +}}} From 5e8d0a773bd6a82d75949ccb53ccdd017ba9cfa8 Mon Sep 17 00:00:00 2001 From: Pierre Siddall <43399998+Pierre-siddall@users.noreply.github.com> Date: Fri, 28 Aug 2026 16:15:03 +0100 Subject: [PATCH 2/4] Add some basic path to MOCI components --- .github/CODEOWNERS | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index b4c9f298..7f6289ab 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -37,9 +37,9 @@ iris ml-avd-support@metoffice.gov.uk {{{ Area Owner rose-stem/ @ericaneininger @Pierre-siddall -Coupled_Drivers @Pierre-siddall @ericaneininger -Postprocessing @ericaneininger -Fortran_Cap +Coupled_Drivers/ @Pierre-siddall @ericaneininger +Postprocessing/ @ericaneininger +Fortran_Cap/ Coupling_Documentation dancopsey }}} @@ -58,10 +58,10 @@ UKESM janemulcahy == Utilities == {{{ Area Owner -moci_lib @ericaneininger +mocilib/ @ericaneininger moci_testing_scripts -dashboard @ericaneininger -ozone_redistribution @ericaneininger +dashboard/ @ericaneininger +ozone_redistribution/ @ericaneininger mean_nemo verification_scripts NGMS_utils @@ -70,10 +70,10 @@ NGMS_utils == Rose meta data == {{{ Area Owner -archive_and_meaning @ericaneininger +rose-meta/archive_and_meaning @ericaneininger coupled_drivers -ocean_ice davestorkey -fcm_make_ocean richardhill +rose-meta/ocean_ice davestorkey +rose-meta/fcm-make-ocean richardhill GA7 GC3 GC3.1 From f87f084b29c023acf63e7b0f475c20f5bb030a6c Mon Sep 17 00:00:00 2001 From: Pierre Siddall <43399998+Pierre-siddall@users.noreply.github.com> Date: Fri, 28 Aug 2026 16:21:27 +0100 Subject: [PATCH 3/4] Make CODEOWNERS file valid --- .github/CODEOWNERS | 143 +++++++++++++++++++++------------------------ 1 file changed, 68 insertions(+), 75 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 7f6289ab..9013f9f1 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,80 +1,73 @@ -= Code Owners = +# = Code Owners = +# +# The purpose of this file is to list ownership of the various components +# held within the MOCI repository. +# +# Please consider the following: +# * Ownership is not a commitment for the individual to undertake all work +# requested or all reviewing for the owned section +# * Any planned changes to a section should be discussed with the code owner +# to ensure plans are appropriate +# * Code owners should be given the option to be a science/technical reviewer +# for any code that alters their section +# * Any interface between components needs to be agreed by all affected parties +# +# Contact details for code owners: +# https://code.metoffice.gov.uk/trac/home/wiki/UserList +# +# Only lines of the form " <@owner> [@owner ...]" are active +# rules. Areas without a known GitHub owner are kept as comments so the +# original ownership table is preserved. -The purpose of this page is to list ownership of the various components -held within the MOCI repository. +# = Coupling Applications: Contacts = +# These are external contacts rather than repository paths, so they are +# recorded here for reference only. +# +# Area Owner +# UM umsysteam@metoffice.gov.uk +# NEMO timgraham +# SI3 davestorkey +# CICE davestorkey +# MEDUSA NOC_southampton +# LFRic timgraham richardhill +# Wave juancastillo +# Rivers dancopsey juancastillo +# OASIS_MCT harryshepherd +# XIOS harryshepherd +# CPMIP +# mule umsysteam@metoffice.gov.uk +# iris ml-avd-support@metoffice.gov.uk -Please consider the following: - * Ownership is not a commitment for the individual to undertake all work - requested or all reviewing for the owned section - * Any planned changes to a section should be discussed with the code owner - to ensure plans are appropriate - * Code owners should be given the option to be a science/technical reviewer - for anyu code that alters their section - * Any interface between components needss to be agreed by all affected parties +# = MOCI Components = +# == Applications == +/rose-stem/ @ericaneininger @Pierre-siddall +/Coupled_Drivers/ @Pierre-siddall @ericaneininger +/Postprocessing/ @ericaneininger +# /Fortran_Cap/ +# Coupling_Documentation dancopsey -Contact details for code owners: https://code.metoffice.gov.uk/trac/home/wiki/UserList +# == Configurations == +# /fcm-make/ +# /Infrastructure_Builds/ +# GC4 +# GC5 +# Global_Coupled_NWP +# UKESM janemulcahy -= Coupling Applications: Contacts = -{{{ -Area Owner -UM umsysteam@metoffice.gov.uk -NEMO timgraham -SI3 davestorkey -CICE davestorkey -MEDUSA NOC_southampton -LFRic timgraham richardhill -Wave juancastillo -Rivers dancopsey juancastillo -OASIS_MCT harryshepherd -XIOS harryshepherd -CPMIP -mule umsysteam@metoffice.gov.uk -iris ml-avd-support@metoffice.gov.uk -}}} +# == Utilities == +/mocilib/ @ericaneininger +/Utilities/dashboard/ @ericaneininger +/Utilities/ozone_redistribution/ @ericaneininger +# moci_testing_scripts +# /Utilities/mean_nemo/ +# /Utilities/verification_scripts/ +# /Utilities/NGMS_utils/ -= MOCI Components = -== Applications == -{{{ -Area Owner -rose-stem/ @ericaneininger @Pierre-siddall -Coupled_Drivers/ @Pierre-siddall @ericaneininger -Postprocessing/ @ericaneininger -Fortran_Cap/ -Coupling_Documentation dancopsey -}}} - - -== Configurations == -{{{ -Area Owner -fcm-make -Infrastucture_Builds -GC4 -GC5 -Global_Coupled_NWP -UKESM janemulcahy -}}} - -== Utilities == -{{{ -Area Owner -mocilib/ @ericaneininger -moci_testing_scripts -dashboard/ @ericaneininger -ozone_redistribution/ @ericaneininger -mean_nemo -verification_scripts -NGMS_utils -}}} - -== Rose meta data == -{{{ -Area Owner -rose-meta/archive_and_meaning @ericaneininger -coupled_drivers -rose-meta/ocean_ice davestorkey -rose-meta/fcm-make-ocean richardhill -GA7 -GC3 -GC3.1 -}}} +# == Rose meta data == +/rose-meta/archive_and_meaning/ @ericaneininger +# /rose-meta/coupled_drivers/ +# /rose-meta/ocean_ice/ davestorkey +# /rose-meta/fcm-make-ocean/ richardhill +# /rose-meta/GA7/ +# /rose-meta/GC3/ +# /rose-meta/GC3.1/ From adb25e8705e3c8da18f6612ed384244128f21e37 Mon Sep 17 00:00:00 2001 From: Pierre Siddall <43399998+Pierre-siddall@users.noreply.github.com> Date: Tue, 1 Sep 2026 11:19:24 +0100 Subject: [PATCH 4/4] Add Richard Hill to LFRic and fcm-make-ocean --- .github/CODEOWNERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 9013f9f1..d222077a 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -29,7 +29,7 @@ # SI3 davestorkey # CICE davestorkey # MEDUSA NOC_southampton -# LFRic timgraham richardhill +/Coupled_Drivers/lfric_driver.py timgraham @HengistPodd # Wave juancastillo # Rivers dancopsey juancastillo # OASIS_MCT harryshepherd @@ -67,7 +67,7 @@ /rose-meta/archive_and_meaning/ @ericaneininger # /rose-meta/coupled_drivers/ # /rose-meta/ocean_ice/ davestorkey -# /rose-meta/fcm-make-ocean/ richardhill +/rose-meta/fcm-make-ocean/ @HengistPodd # /rose-meta/GA7/ # /rose-meta/GC3/ # /rose-meta/GC3.1/