-
Notifications
You must be signed in to change notification settings - Fork 6.7k
Add MCPReview #1981
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add MCPReview #1981
Conversation
Hi @henroger thanks for the PR. I took a look at the link and I don't see a way to view or submit reviews as mentioned in the description you added here. Could you share more info on whether this is actually accurate? |
Hi @olaservo we do have review score and comments now and will be adding AI MCP review report soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, will grab this in the next round of Resource section updates.
@olaservo hi, I am sorry to reactivate this PR again, just wondering is this ever going to be merged? |
Hi @henroger sorry and thanks for the ping, this one was going to go through with an aggregated PR but I decided to merge the non-server PRs separately instead. This has been merged. |
Description
Server Details
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context