Skip to content

Conversation

@jonathanhefner
Copy link
Member

Adds the MCP_APPS_SDK role for jonathanhefner to grant admin access to the ext-apps repository.

@jonathanhefner jonathanhefner enabled auto-merge (squash) January 19, 2026 13:50
"resolved": "https://registry.npmjs.org/@opentelemetry/api/-/api-1.9.0.tgz",
"integrity": "sha512-3giAOQvZiH5F9bMlMiv8+GSPMeqg0dbaeo58/0SlA9sxSqZhnUtxzX9/2FzyhS9sWQf5S0GJE0AKBrFqjpeYcg==",
"license": "Apache-2.0",
"peer": true,
Copy link
Member

Choose a reason for hiding this comment

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

hmm these seem unrelated, can you remove? worried they'll flip back and forth

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 is the result of running npm install && npm run format in order to fix the build: d504f54.

I will remove that commit though, and just format my entry.

Copy link
Member

@pcarleton pcarleton Jan 21, 2026

Choose a reason for hiding this comment

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

yea makes sense, iirc it depends on your npm version, so if someone else runs it on a different version, it will go remove those 'peer' lines. i think the solution is to pin the npm version, but let's tackle that separately

Copy link
Member

@pcarleton pcarleton left a comment

Choose a reason for hiding this comment

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

package-lock.json

Adds the `MCP_APPS_SDK` role for `jonathanhefner` to grant admin access
to the `ext-apps` repository.
@jonathanhefner jonathanhefner force-pushed the add-jonathanhefner-to-mcp_apps_sdk branch from d504f54 to 251d17a Compare January 21, 2026 13:40
@jonathanhefner jonathanhefner merged commit ee35c86 into modelcontextprotocol:main Jan 21, 2026
1 of 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