Skip to content

Update mcp extension #18932

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update mcp extension #18932

wants to merge 1 commit into from

Conversation

shkm
Copy link
Contributor

@shkm shkm commented May 3, 2025

Description

Initially I noticed that this was broken under fish. The -L argument seems to be ignored by zsh but errors in bash, and also errors in fish.

This change uses the same shell execution args for fetching env data in both cases.

Screencast

Checklist

- Use login shell for fetching env data
- Initial commit
@raycastbot
Copy link
Collaborator

Thank you for your contribution! 🎉

🔔 @EvanZhouDev @radzio you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: mcp Issues related to the mcp extension labels May 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension fix / improvement Label for PRs with extension's fix improvements extension: mcp Issues related to the mcp extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants