Skip to content

Add organization and email for Matthew Bain - #306

Closed
rocketstack-matt wants to merge 1 commit into
mainfrom
rocketstack-matt-patch-1
Closed

rocketstack-matt wants to merge 1 commit into
mainfrom
rocketstack-matt-patch-1

Conversation

@rocketstack-matt

Copy link
Copy Markdown
Member

Updated Matthew Bain's entry to include organization and email.

Updated Matthew Bain's entry to include organization and email.
Copilot AI review requested due to automatic review settings July 24, 2026 10:34

Copilot AI left a comment

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.

Pull request overview

Updates the repository鈥檚 maintainer roster by completing missing contact details for an existing maintainer entry, keeping the MAINTAINERS list accurate and actionable for project governance.

Changes:

  • Populated Matthew Bain鈥檚 maintainer row with organization and email.

馃挕 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread MAINTAINERS.md
| @maria-mcparland | Maria McParland | *please add organization* | *please add email* |
| @psmulovics | Peter Smulovics | Morgan Stanley | *please add email* |
| @rocketstack-matt | Matthew Bain | *please add organization* | *please add email* |
| @rocketstack-matt | Matthew Bain | Morgan Stanley | matt@rocketstack.co |

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.

@rocketstack-matt If we add our emails here are we opening ourselves up to a load of spam emails? Wondering if we want to remove our emails and just have the TOC mail

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is the new standard for FINOS repos - @TheJuanAndOnly99 thoughts?

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.

Generally we try to avoid sharing emails, even in the other repos that has the new MAINTAINERS.md - as there is usually a maintainers mailgroup people can write to.

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