NotAShelf / nvf

A highly modular, extensible and distro-agnostic Neovim configuration framework for Nix/NixOS.
https://notashelf.github.io/nvf/
MIT License
158 stars 26 forks source link

Code does not reflect documentation regarding `programs.nvf` #279

Closed Nowaaru closed 4 months ago

Nowaaru commented 4 months ago

⚠️ Please verify that this bug has NOT been reported before.

Description

In the main and v0.6 branch, the code describes "programs.neovim-flake" whereas the documentation describes "programs.nvf"

screenshot (lightmode zzz) ![image](https://github.com/NotAShelf/nvf/assets/16274568/dd4eb992-9e99-4567-81fa-a6677f925e25)

👟 Reproduction steps

N/A

👀 Expected behavior

A warning trace appearing when attempting to index neovim-flake before redirecting to programs.nvf.

😓 Actual Behavior

error: The option 'programs.nvf' does not exist.

💻 Metadata

📝 Relevant log output

error: The option `programs.nvf' does not exist. Definition values:
       - In `/nix/store/aafrhbikr2p3mz3zazhx89br0whaf1g9-source/usr/dev/vim.nix':
           {
             enable = true;
             settings = { };
           }
NotAShelf commented 4 months ago

Thanks for reporting this, I somehow managed to skip renaming NixOS and Home-Manager modules while I was going through renaming the project.

Pushed a quick patch to the rename-module-opts branch, #280 will have the details. Would you be able to test the rename-module-opts branch and tell me if it works as intended?

Nowaaru commented 4 months ago

Getting a 'renaming error' when indexing programs.nvf though I never index neovim-flake. Trace is attached. hm.log

NotAShelf commented 4 months ago

Minor mistake on my behalf, could you try again please?

Nowaaru commented 4 months ago

Looks good. Good stuff, Shelf-man. 👍

NotAShelf commented 4 months ago

Thank you for testing :)