Skip to content

Support disabling schema source URI in hovers#1242

Merged
datho7561 merged 3 commits intoredhat-developer:mainfrom
shin19991207:fix-1120
Apr 30, 2026
Merged

Support disabling schema source URI in hovers#1242
datho7561 merged 3 commits intoredhat-developer:mainfrom
shin19991207:fix-1120

Conversation

@shin19991207
Copy link
Copy Markdown
Member

@shin19991207 shin19991207 commented Apr 29, 2026

What does this PR do?

Added the yaml.hoverSchemaSource configuration option to control whether the schema source is shown in hover tooltips. Defaults to true.

What issues does this PR fix or reference?

Fixes #1120

Is it tested? How?

  • Tested manually
  • Automated tests

@shin19991207 shin19991207 requested a review from datho7561 as a code owner April 29, 2026 22:18
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Seems to be working well, just think the name could be better.

You might want to create a PR in vscode-yaml in order to register the setting there so it shows up in the settings page. As a convention, when we add new settings, we generally open a PR with the setting registered on the client side at the same time.

Comment thread src/languageserver/handlers/settingsHandlers.ts Outdated
@shin19991207
Copy link
Copy Markdown
Member Author

Created a PR in vscode-yaml to register the setting: redhat-developer/vscode-yaml#1238

Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks, Morgan!

Signed-off-by: Morgan Chang <shin19991207@gmail.com>
Signed-off-by: Morgan Chang <shin19991207@gmail.com>
Signed-off-by: Morgan Chang <shin19991207@gmail.com>
@datho7561 datho7561 merged commit 46328b6 into redhat-developer:main Apr 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Disable "Source" URI in Hovers

2 participants