Skip to content

chore: Add MAINTAINERS.md file - #1009

Open
TheJuanAndOnly99 wants to merge 12 commits into
mainfrom
add-maintainers-file
Open

chore: Add MAINTAINERS.md file#1009
TheJuanAndOnly99 wants to merge 12 commits into
mainfrom
add-maintainers-file

Conversation

@TheJuanAndOnly99

@TheJuanAndOnly99 TheJuanAndOnly99 commented May 25, 2026

Copy link
Copy Markdown
Member

This PR adds a MAINTAINERS.md file to the root of this repository as part of a FINOS-wide effort to standardize maintainer signaling across all FINOS projects. An announcement was sent to community@finos.org with the full context, see announcement email here.

How this list was generated

The maintainers were determined from this repository's GitHub settings:

  • Users with maintain or admin role on the repository
  • Members of teams with maintain or admin permission

Each entry includes the GitHub username, name, and organization pulled from the user's public GitHub profile. Where name or organization is missing, a *please add ...* placeholder marks where to fill in the value. Email is optional: it is included when the user has a public GitHub email, otherwise the cell is left blank.

If no maintainers were detected automatically (e.g. your repo has no users or teams with maintain/admin role), the table is intentionally empty. Please populate it manually on this branch before merging and email help@finos.org.

What we need from this project's maintainers

  1. Review the list for completeness and accuracy.
  2. Update any incorrect or placeholder values (names, organization) directly on this branch. Email is optional.
  3. Add anyone missing, edit the file in this PR if a maintainer was not automatically detected.
  4. Merge when the list is accurate.

If your project does not need a MAINTAINERS.md for any reason, just close this PR and let us know at help@finos.org.

If this repository should instead be archived (e.g. it is no longer actively maintained or has been superseded), please email help@finos.org instead so we can coordinate the archival with you.

If you already have a MAINTAINERS.md file, you can close this PR. Please update your existing file if needed and place it in the root of the repository.

Multi-repo projects: share one MAINTAINERS.md

If this repository is part of a project with multiple repositories that share (and will continue to share) the same maintainers, you don't need a separate list per repo. Pick one repository (typically the project's main/umbrella repo) to host the canonical MAINTAINERS.md, and on every other repo replace the table in this PR with a short note pointing readers there, for example:

The maintainers of this repository are listed in the main project repository. See list here.

That way maintainer changes only need to be made in one place going forward.

Going forward: keeping the list current

To keep maintainer changes transparent and aligned with open governance:

  • Any change to the maintainer list must be made via a PR to this file.
  • If your project's governance requires a vote, document or link to the vote outcome in the PR description or comments.
  • This gives the community a public audit trail of project leadership over time.

If you have any questions about the format or process, please email help@finos.org.

Thank you for your review!

Signed-off-by: Juan Estrella <juan.estrella@finos.org>
@TheJuanAndOnly99
TheJuanAndOnly99 requested a review from a team as a code owner May 25, 2026 21:45
@netlify

netlify Bot commented May 25, 2026

Copy link
Copy Markdown

Deploy Preview for common-cloud-controls canceled.

Name Link
🔨 Latest commit 1ccbc04
🔍 Latest deploy log https://app.netlify.com/projects/common-cloud-controls/deploys/6a84c441d49a2f0008395c4a

Comment thread MAINTAINERS.md Outdated
@robmoffat
robmoffat self-requested a review May 28, 2026 16:12
zigmax and others added 3 commits June 4, 2026 10:19
Co-authored-by: sshiells-scottlogic <148051590+sshiells-scottlogic@users.noreply.github.com>
@TheJuanAndOnly99

Copy link
Copy Markdown
Member Author

Hi @eddie-knight can you help get this reviewed and merged? We don't necessarily need all the details (especially email) to be present before merging as long as the list of names + github usernames is correct and up to date we can marge and update the other fields later as needed.

Comment thread MAINTAINERS.md Outdated
Comment on lines +9 to +21
| @Alexstpierrework | Alex St. Pierre | *please add organization* | *please add email* |
| @aric-rosenbaum | Aric Rosenbaum | @redhat | *please add email* |
| @ececon | *please add name* | *please add organization* | *please add email* |
| @eddie-knight | Eddie Knight | *please add organization* | *please add email* |
| @jared-lambert | Jared Lambert | Microsoft | *please add email* |
| @jpower432 | Jennifer Power | Red Hat | jpower@redhat.com |
| @lucaborella89 | Luca Borella | *please add organization* | *please add email* |
| @mlysaght2017 | Michael Lysaght | *please add organization* | *please add email* |
| @rgriffiths-scottlogic | Robert Griffiths | @ScottLogic | *please add email* |
| @smendis-scottlogic | Sonali Mendis | Scott Logic | *please add email* |
| @sshiells-scottlogic | Stevie Shiells | Scott Logic | sshiells@scottlogic.com |
| @sublimino | Andrew Martin | @controlplaneio | *please add email* |
| @zigmax | Maxime Coquerel | Royal Bank of Canada (RBC) | max.coquerel@gmail.com |

@eddie-knight eddie-knight Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| @Alexstpierrework | Alex St. Pierre | *please add organization* | *please add email* |
| @aric-rosenbaum | Aric Rosenbaum | @redhat | *please add email* |
| @ececon | *please add name* | *please add organization* | *please add email* |
| @eddie-knight | Eddie Knight | *please add organization* | *please add email* |
| @jared-lambert | Jared Lambert | Microsoft | *please add email* |
| @jpower432 | Jennifer Power | Red Hat | jpower@redhat.com |
| @lucaborella89 | Luca Borella | *please add organization* | *please add email* |
| @mlysaght2017 | Michael Lysaght | *please add organization* | *please add email* |
| @rgriffiths-scottlogic | Robert Griffiths | @ScottLogic | *please add email* |
| @smendis-scottlogic | Sonali Mendis | Scott Logic | *please add email* |
| @sshiells-scottlogic | Stevie Shiells | Scott Logic | sshiells@scottlogic.com |
| @sublimino | Andrew Martin | @controlplaneio | *please add email* |
| @zigmax | Maxime Coquerel | Royal Bank of Canada (RBC) | max.coquerel@gmail.com |
| @aric-rosenbaum | Aric Rosenbaum | Red Hat | |
| @eddie-knight | Eddie Knight | Revanite | |
| @jpower432 | Jennifer Power | Red Hat | |
| @mlysaght2017 | Michael Lysaght | Citi | |
| @smendis-scottlogic | Sonali Mendis | Scott Logic | |
| @sshiells-scottlogic | Stevie Shiells | Scott Logic | |
| @zigmax | Maxime Coquerel | Royal Bank of Canada (RBC) | |

@aric-rosenbaum @sshiells-scottlogic we're overdue for a revamp on the outdated governance docs, and I think the README is out of date as well. We have avoided using the term "maintainer" in the past, favoring things like "WG Lead" and "Steerco Member," but I suspect we can't stay on that track.

Here's the list I submitted on the fly for the TOC presentation yesterday, focusing on technical contributors since we have any updated list of maintainers:

  • Eddie Knight (Revanite)
  • Rob Moffat (FINOS)
  • Sonali Mendis (Scott Logic)
  • Stevie Shiells (Scott Logic)
  • Michael Lysaght (Citi)
  • Leroy Abhikui (LSEG)
  • Dave Reeve (Morgan Stanley)
  • Jenn Power (Red Hat)
  • Maxime Coquerel (RBC)
  • Sankara Ramakrishnan (Black Rock)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@eddie-knight do you want to raise an issue to update the docs?

I can get the Scott Logic folks working on the website to review the content and they may be able to make some updates based on the new content on the website.

@gbalysevaite FYI

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Added #1158

zigmax and others added 4 commits July 24, 2026 20:40
Drop the please-add-email placeholder so a missing address is no longer treated as required. The Email column stays; existing addresses are left unchanged.

Signed-off-by: Juan Estrella <36825759+TheJuanAndOnly99@users.noreply.github.com>
Drop the please-add-email placeholder and label the column Email (optional). Existing addresses are left unchanged.

Signed-off-by: Juan Estrella <36825759+TheJuanAndOnly99@users.noreply.github.com>
@zigmax
zigmax marked this pull request as ready for review August 18, 2026 00:48
@eddie-knight eddie-knight changed the title Add MAINTAINERS.md file chore: Add MAINTAINERS.md file Aug 18, 2026
Comment thread MAINTAINERS.md Outdated
Co-authored-by: Eddie Knight <21176439+eddie-knight@users.noreply.github.com>
@eddie-knight
eddie-knight enabled auto-merge (squash) August 18, 2026 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants