Skip to content

Fix default value of two extension settings#1247

Merged
datho7561 merged 1 commit intoredhat-developer:mainfrom
rchl:patch-1
May 8, 2026
Merged

Fix default value of two extension settings#1247
datho7561 merged 1 commit intoredhat-developer:mainfrom
rchl:patch-1

Conversation

@rchl
Copy link
Copy Markdown
Contributor

@rchl rchl commented May 8, 2026

The type of those properties is boolean so the default should be literal boolean and not a string.

What does this PR do?

fix extension schema definition

What issues does this PR fix or reference?

No issue

Is it tested? How?

validating my settings (in sublime text) against schema)

The type of those properties is boolean so the default should be literal boolean and not a string.
@rchl rchl requested a review from datho7561 as a code owner May 8, 2026 06:24
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, Rafał!

@datho7561 datho7561 added this to the 1.24.0 milestone May 8, 2026
@datho7561 datho7561 added the bug label May 8, 2026
@datho7561 datho7561 merged commit 3d5bfd8 into redhat-developer:main May 8, 2026
5 checks passed
@rchl rchl deleted the patch-1 branch May 8, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants