Skip to content

fix: improve/extend LuaLS annotations, code greatly optimized#2

Merged
aikhe merged 1 commit intoaikhe:masterfrom
DrKJeff16:patch/fixup
Feb 15, 2026
Merged

fix: improve/extend LuaLS annotations, code greatly optimized#2
aikhe merged 1 commit intoaikhe:masterfrom
DrKJeff16:patch/fixup

Conversation

@DrKJeff16
Copy link
Contributor

@DrKJeff16 DrKJeff16 commented Feb 15, 2026

Description

Hello there, developer of triforce.nvim here. I took a look at your project and it's quite lovely!

I wanted to improve development by extending all possible LuaLS type annotations.
I also made some minimal code optimizations.

Please, if there's any feedback let me know! volt can be a bit of a pain to work with IMO because of the variably-nested lists...

Sorry for such a big PR btw!

Signed-off-by: Guennadi Maximov C <g.maxc.fox@protonmail.com>
Copy link
Contributor Author

@DrKJeff16 DrKJeff16 left a comment

Choose a reason for hiding this comment

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

For clarification I converted this into a function because of the danger of users being able to change the defaults indirectly.
Setting this as a return value guarantees that the default values won't run the risk of accidental changes.

@aikhe
Copy link
Owner

aikhe commented Feb 15, 2026

yo, big fan! agreed, having no docs about volt and it's API is a struggle.

For clarification I converted this into a function because of the danger of users being able to change the defaults indirectly.
Setting this as a return value guarantees that the default values won't run the risk of accidental changes.

good to know, appreciate the help!

@aikhe aikhe merged commit a4309d3 into aikhe:master Feb 15, 2026
@aikhe aikhe self-assigned this Feb 15, 2026
@aikhe aikhe self-requested a review February 15, 2026 14:25
@DrKJeff16 DrKJeff16 deleted the patch/fixup branch February 15, 2026 17:02
@DrKJeff16
Copy link
Contributor Author

@aikhe I'm working on an annotated Pull Request for volt. I believe it will help you with development.

nvzone/volt#15

@aikhe
Copy link
Owner

aikhe commented Feb 16, 2026

noice

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