-
Notifications
You must be signed in to change notification settings - Fork 8
initial http mode (#6) #7
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
base: main
Are you sure you want to change the base?
Conversation
323af62
to
5854480
Compare
README.md
Outdated
"x-bitrise-enabled-api-groups: cache-items,pipelines" | ||
], | ||
"env": { | ||
"PATH": "/.../..../nodejs/20.13.1/bin:/bin" |
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.
This won't be the same for everyone, right? In that case I would remove it and make a comment that you might need to pass an absolute path to command
or set the PATH
env
var accordingly.
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.
I actually documented uvx
like <absolute path to>/uvx
bit it was removed for some reason.
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.
Also, we should probably document how one can run the MCP server in HTTP mode as well in this section to paint the full picture.
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.
Also document that we have mcp.bitrise.io
.
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.
Left some nitpicks and a question, but LGTM! 🙂🎉
Co-authored-by: Szabolcs Gelencsér <74185105+szabolcsgelencser@users.noreply.github.com>
No description provided.