Skip to content

feat(diagnostics): add wrong/empty enum Value diagnostic#809

Open
cw-1988 wants to merge 1 commit into
nix-community:mainfrom
cw-1988:feat/805
Open

feat(diagnostics): add wrong/empty enum Value diagnostic#809
cw-1988 wants to merge 1 commit into
nix-community:mainfrom
cw-1988:feat/805

Conversation

@cw-1988

@cw-1988 cw-1988 commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

#805, completions on top are added in #810

AI slopped with GPT-5.4 High, but a basic DX feature i had to add.

Take your time, read at your own pace, this issue isnt going anywhere.

I will use my own fork with the features i deem necessary. I'm just here to share the goods. Make all the edits you want, or give me the necessary feedback so i can improve upon this PR.

I hope you are doing well and have a nice day ;-)

@cw-1988 cw-1988 requested a review from inclyc as a code owner April 12, 2026 07:34

@inclyc inclyc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR only validates whether enum values exist within the type system, but it skips other types in lib.type like bool or string. I'm concerned that checking only enums might be a bit 'heavy' in terms of performance impact. What do you think?

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.

2 participants