Skip to content
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

Fix: Commented out MCP servers because they were causing issues, preventing the services from running #6316

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

Conversation

thecodingwizardx
Copy link

Pull Request Template

⚠️ Before Submitting a PR, Please Review:

  • Please ensure that you have thoroughly read and understood the Contributing Docs before submitting your Pull Request.

⚠️ Documentation Updates Notice:

  • Kindly note that documentation updates are managed in this repository: librechat.ai

Summary

Please provide a brief summary of your changes and the related issue. Include any motivation and context that is relevant to your changes. If there are any dependencies necessary for your changes, please list them here.

Change Type

Please delete any irrelevant options.

  • Bug fix (non-breaking change which fixes an issue)

Testing

Please describe your test process and include instructions so that we can reproduce your test. If there are any important variables for your testing configuration, list them here.

Test Configuration:

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • My changes do not introduce new warnings

@rubentalstra
Copy link
Collaborator

this is not a fix

@rubentalstra rubentalstra added the 🚫 wontfix This will not be worked on label Mar 12, 2025
Copy link
Author

@thecodingwizardx thecodingwizardx left a comment

Choose a reason for hiding this comment

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

it is a fix, without commenting that, its not running. its throwing lot of errors.
confirmed by @fuegovic

@fuegovic
Copy link
Collaborator

@rubentalstra the librechat.yaml example file should be a valid file imo, it's just confusing for newcomers otherwise

@rubentalstra
Copy link
Collaborator

@fuegovic okay I understand. I will reopen it then. sorry about that. only seeing the code did the PR not make sense 😅

@rubentalstra rubentalstra reopened this Mar 12, 2025
@rubentalstra rubentalstra added 🔧 fix Bug fixes and code patches and removed 🚫 wontfix This will not be worked on labels Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 fix Bug fixes and code patches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants