Skip to content

Expose base type of enumeration#39

Open
rchl wants to merge 1 commit intomainfrom
feat/enumeration-base-type
Open

Expose base type of enumeration#39
rchl wants to merge 1 commit intomainfrom
feat/enumeration-base-type

Conversation

@rchl
Copy link
Copy Markdown
Member

@rchl rchl commented Apr 20, 2026

Enumerations that specify supportsCustomValues now get the base type (str or int) included where those are referenced.

It solves the use case that #37 was solving but in a different way.

Note that root_symbol_name was unused.

@rchl rchl force-pushed the feat/enumeration-base-type branch from 7aa44a7 to a057c2f Compare April 20, 2026 20:22
Comment thread generated/lsp_types.py
@since 3.17.0 support for relative patterns.
"""
kind: NotRequired['WatchKind']
kind: NotRequired[Union[Uint, WatchKind]]
Copy link
Copy Markdown
Member Author

@rchl rchl Apr 20, 2026

Choose a reason for hiding this comment

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

This is potentially controversial or even wrong. Since we use IntFlag for this one, we don't really need to allow int value for it.

Though from the spec perspective it is probably correct because the value is a bitwise flag so not necessarily the exact value that matches one of the enums.

@rchl rchl changed the title Feat/enumeration base type Expose base type of enumeration Apr 20, 2026
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.

1 participant