mirror of
https://github.com/nix-community/nixvim.git
synced 2025-06-21 00:25:42 +02:00
docs: don't remove freeform sub-options
This isn't a fully correct fix, but it's much closer to the intended behaviour.
This commit is contained in:
parent
19528509e5
commit
9caeb51238
1 changed files with 13 additions and 1 deletions
|
@ -24,7 +24,19 @@ let
|
||||||
|
|
||||||
removeUnwanted =
|
removeUnwanted =
|
||||||
attrs:
|
attrs:
|
||||||
builtins.removeAttrs attrs [
|
# FIXME: We incorrectly remove _freeformOptions here.
|
||||||
|
#
|
||||||
|
# However we can't fix the bug because we derive page names from attrnames;
|
||||||
|
# the correct behaviour would be to ignore attrnames and use option locs.
|
||||||
|
#
|
||||||
|
# As a workaround, merge the freeform options at the top of these attrs,
|
||||||
|
# however this can run into name conflicts 😢
|
||||||
|
#
|
||||||
|
# We should move this workaround to where we decide the page name and
|
||||||
|
# whether to nest into a sub-page, so that we can keep the original
|
||||||
|
# _freeformOptions attr as intended.
|
||||||
|
attrs._freeformOptions or { }
|
||||||
|
// builtins.removeAttrs attrs [
|
||||||
"_module"
|
"_module"
|
||||||
"_freeformOptions"
|
"_freeformOptions"
|
||||||
"warnings"
|
"warnings"
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue