You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am just setting a pre_hook option, to override the default nil. But the lua diagnostics are warning about Missing required fields in type CommentConfig.
The docs mention that this should be merged with the default config, I'm wondering if there needs to be a DTO type that marks those fields as optional for the setup function options so they can be merged without warnings. Im not to familiar with lua types though 😅
Following are the default config for the setup(). If you want to override, just modify the option that you want then it will be merged with the default config. Read :h comment.config for more info.
If this would be a medium difficulty fix I would be willing to make a PR if your are accepting contributions!
The text was updated successfully, but these errors were encountered:
I am just setting a pre_hook option, to override the default
nil
. But the lua diagnostics are warning aboutMissing required fields in type CommentConfig
.The docs mention that this should be merged with the default config, I'm wondering if there needs to be a DTO type that marks those fields as optional for the setup function options so they can be merged without warnings. Im not to familiar with lua types though 😅
If this would be a medium difficulty fix I would be willing to make a PR if your are accepting contributions!
The text was updated successfully, but these errors were encountered: