From 94026239382f5e4ffba09587dbe58449de4e9b61 Mon Sep 17 00:00:00 2001
From: "Seth R. Johnson" <johnsonsr@ornl.gov>
Date: Mon, 27 Jan 2025 16:23:58 -0500
Subject: [PATCH] Update core advisors (#1598)

* Remove Demarteau and Romano

* Add a note recommending reevaluating at the beginning of the year

* Work around https://github.com/actions/checkout/issues/2041
---
 .github/workflows/doc.yml               | 4 ++++
 doc/_static/ornltm-header-celeritas.tex | 6 ------
 doc/conf.py                             | 2 --
 doc/development/administration.rst      | 7 ++++++-
 4 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/.github/workflows/doc.yml b/.github/workflows/doc.yml
index 2ad9e93c5e..665e6ce4f8 100644
--- a/.github/workflows/doc.yml
+++ b/.github/workflows/doc.yml
@@ -37,6 +37,8 @@ jobs:
         # Turn warnings into errors only for PRs
         # Disable expensive graphs for PRs also
         run: |
+          # NOTE: tags have issues, see https://github.com/actions/checkout/issues/2041
+          git fetch --tags
           ln -fs scripts/cmake-presets/ci-ubuntu-github.json CMakeUserPresets.json
           mkdir build && cd build
           cmake --preset=${CMAKE_PRESET} --log-level=VERBOSE \
@@ -83,6 +85,8 @@ jobs:
             pip install -r scripts/doc-requirements.txt
       - name: Configure celeritas
         run: |
+          # NOTE: tags have issues, see https://github.com/actions/checkout/issues/2041
+          git fetch --tags
           ln -fs scripts/cmake-presets/ci-ubuntu-github.json CMakeUserPresets.json
           mkdir build && cd build
           cmake --preset=${CMAKE_PRESET} --log-level=VERBOSE \
diff --git a/doc/_static/ornltm-header-celeritas.tex b/doc/_static/ornltm-header-celeritas.tex
index bf16d567ee..0c56d11291 100644
--- a/doc/_static/ornltm-header-celeritas.tex
+++ b/doc/_static/ornltm-header-celeritas.tex
@@ -28,13 +28,7 @@
   Philippe Canal\affilnum{2}
   %%
   \and
-  Marcel Demarteau\affilnum{1}
-  %%
-  \and
   Thomas Evans\affilnum{1}
-  %%
-  \and
-  Paul Romano\affilnum{3}
 }%
 
 \affiliation{%
diff --git a/doc/conf.py b/doc/conf.py
index 9c720860d1..dd7a3540ce 100644
--- a/doc/conf.py
+++ b/doc/conf.py
@@ -26,8 +26,6 @@
  'Stefano C Tognini',
  # Core advisors
  'Thomas M Evans',
- 'Marcel Demarteau',
- 'Paul Romano',
 ]
 author = " and ".join(all_authors)
 copyright = '{:%Y}, UT–Battelle/ORNL and Celeritas team'.format(
diff --git a/doc/development/administration.rst b/doc/development/administration.rst
index 4be90a5acc..ecf2a40303 100644
--- a/doc/development/administration.rst
+++ b/doc/development/administration.rst
@@ -76,7 +76,12 @@ Contributor
 Role change process
 -------------------
 
-Adding or removing a member of the "core team" must be done by consensus of the
+Roles should be periodically reevaluated to reflect current participation in
+the project. It is recommended to re-evaluate core roles at the start of each
+calendar year based on the last year's participation and upcoming work plans.
+
+Adding or removing a member of the "core team" can be done at any time by
+consensus of the
 leadership team (or if the core team member wants to remove themself). Adding
 maintainers can be done at the whim of the code lead. The `team list`_ on
 GitHub is the official record of roles.