Repository navigation
Conversation
|
So I figured out how to do what I want to do without this PR. If you want it anyways, I could still keep it open, though? Your call. :) |
|
Hey, the CI went through some changes to make sure it works for older versions of Linux. Please merge the changes from master to your branch and re-run the CI (I might need to re-approve it) |
This allows to have nested tables for variables.
|
Done! (Edit: And yeah, looks like you need to re-approve) |
| } | ||
| } | ||
|
|
||
| /// Merge two TOML tables |
There was a problem hiding this comment.
This function already exists:
Line 179 in 1712b25
You should either rewrite the below code to use it or rewrite the code that uses that function to use this instead
There was a problem hiding this comment.
Oh wow, I'm sorry, how did I not notice that? Will do, thank you!
|
I created a fix for this issue. Is there any way I can add to this pull request, or do I have to create a new one? |
I think you need to create a new one. For bonus points, add a test that fails without the fix and passes with the fix :) |
|
Closed, see #102 |
3 participants
This allows to have nested tables for variables.
I'm using it in my Neovim config to do some package-specific configuration if (and only if) a package is enabled:
With this, I can loop over the
neovim.subpackagesvariable and do something for all enabled packages only鈥攍ike run some package-specific code:Now, if I decide to disable the
luadevpackage, I won't have to changesubpackages.luain any way鈥攕ince the variableneovim.subpackages.luadevdoesn't exist anymore, it won't be in thesubpackageslist.I'm pretty sure this would enable other functionality as well, considering the power of the handlebars templating facility.