-
Notifications
You must be signed in to change notification settings - Fork 4.1k
fix: merge yaml.schemas settings #9483
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
Conversation
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
1 similar comment
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
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.
No issues found across 1 file
9d3e47a to
8790b8e
Compare
I don't think this error is related to my PR :/ |
Users may already have `yaml.schemas` preferences and the extension should not be overwriting user settings. The code is however free to control keys for `extensionUri`, as long as it leaves other entries alone. See also continuedev#7080
8790b8e to
a0729bd
Compare
|
🎉 This PR is included in version 1.39.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |

Description
Users may already have
yaml.schemaspreferences and the extension should not be overwriting user settings.The code is however free to control keys for
extensionUri, as long as it leaves other entries alone.See also #7080
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
N/A
Tests
N/A
Continue Tasks
Powered by Continue
Summary by cubic
Preserve user-defined yaml.schemas settings by merging instead of overwriting when registering the extension’s schema. The extension only adds/updates its own entry keyed by extensionUri and leaves other schema mappings unchanged.
Written for commit a0729bd. Summary will update on new commits.