Skip to content

Conversation

@idosal
Copy link
Contributor

@idosal idosal commented Jan 27, 2026

No description provided.

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

Copilot AI left a comment

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 on lines 269 to 273
{
github: 'idosal',
discord: '593070927202484244',
memberOf: [ROLE_IDS.WORKING_GROUPS, ROLE_IDS.MCP_APPS_WG, ROLE_IDS.MCP_APPS_SDK],
},
Copy link

Copilot AI Jan 27, 2026

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
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.

2 participants