Skip to content

Add Ido Salomon to maintainers - #46

Merged
dsp-ant merged 2 commits into
modelcontextprotocol:mainfrom
idosal:patch-1
Jan 28, 2026
Merged

dsp-ant merged 2 commits into
modelcontextprotocol:mainfrom
idosal:patch-1

Conversation

@idosal

@idosal idosal commented Jan 27, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 27, 2026 20:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a new member "Ido Salomon" (GitHub username: idosal) to the maintainers list with roles in working groups and MCP Apps.

Changes:

  • Added a new member entry for 'idosal' with Discord ID and role assignments to WORKING_GROUPS, MCP_APPS_WG, and MCP_APPS_SDK

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

Comment thread src/config/users.ts Outdated
Comment on lines +269 to +273
{
github: 'idosal',
discord: '593070927202484244',
memberOf: [ROLE_IDS.WORKING_GROUPS, ROLE_IDS.MCP_APPS_WG, ROLE_IDS.MCP_APPS_SDK],
},

Copilot AI Jan 27, 2026

Copy link

Choose a reason for hiding this comment

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

The entry for 'idosal' is not in alphabetical order. The MEMBERS array is sorted alphabetically by GitHub username. This entry should be placed after 'ihrpr' (currently at line 200) and before 'jamadeo' (currently at line 204), not after 'kurtisvg'.

Copilot uses AI. Check for mistakes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

@dsp-ant
dsp-ant self-requested a review January 28, 2026 14:51
dsp-ant
dsp-ant previously approved these changes Jan 28, 2026
@dsp-ant
dsp-ant enabled auto-merge (squash) January 28, 2026 15:13
@dsp-ant
dsp-ant disabled auto-merge January 28, 2026 15:14
@dsp-ant
dsp-ant merged commit d06127f into modelcontextprotocol:main Jan 28, 2026
2 checks passed
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.

3 participants