Skip to content

Add Bob Dickinson to Inspector Maintainers#64

Merged
dsp merged 1 commit intomodelcontextprotocol:mainfrom
cliffhall:add-bob-dickinson
Feb 16, 2026
Merged

Add Bob Dickinson to Inspector Maintainers#64
dsp merged 1 commit intomodelcontextprotocol:mainfrom
cliffhall:add-bob-dickinson

Conversation

@cliffhall
Copy link
Member

@cliffhall cliffhall commented Feb 16, 2026

Description

In users.ts

  • add ROLE_IDS.MAINTAINERS and ROLE_IDS.INSPECTOR_MAINTAINERS to Bob Dickinson's entry

Motivation and Context

The Inspector maintainers group voted to add Bob Dickinson.

How Has This Been Tested?

Ran validate and check

Screenshot 2026-02-16 at 11 00 14 AM Screenshot 2026-02-16 at 11 00 29 AM

Breaking Changes

Nope.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • User roles update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

From a vote in Discord: https://discord.com/channels/1358869848138059966/1471250680969822230

  - add ROLE_IDS.MAINTAINERS and ROLE_IDS.INSPECTOR_MAINTAINERS to Bob Dickinson's entry as per DSP
@cliffhall cliffhall requested a review from a team as a code owner February 16, 2026 16:02
memberOf: [ROLE_IDS.SKILLS_OVER_MCP_IG],
email: 'bob.dickinson@gmail.com',
discord: '1175893001202045139',
memberOf: [ROLE_IDS.MAINTAINERS, ROLE_IDS.INSPECTOR_MAINTAINERS, ROLE_IDS.SKILLS_OVER_MCP_IG],
Copy link
Member

Choose a reason for hiding this comment

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

Is the SKILLS_OVER_MCP_IG correct here?

Copy link
Member Author

Choose a reason for hiding this comment

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

It was already there. Not certain what the role by itself will allow or if it also needs ROLE_IDS.INTEREST_GROUPS but there are some (like caitiem20 a couple below this one) that has just a working group.

@dsp dsp enabled auto-merge (squash) February 16, 2026 16:27
@dsp dsp self-requested a review February 16, 2026 16:28
@dsp dsp merged commit 182324d into modelcontextprotocol:main Feb 16, 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