Skip to content

docs: nix package instead of custom fetch - #27

Merged
sand4rt merged 1 commit into
mainfrom
docs-nix-package
Aug 23, 2026
Merged

docs: nix package instead of custom fetch#27
sand4rt merged 1 commit into
mainfrom
docs-nix-package

Conversation

@sand4rt

@sand4rt sand4rt commented Aug 23, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 23, 2026 13:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The updated Nix snippet uses unquoted attribute access for a hyphenated name (zen-nvim), which will fail to evaluate and should be corrected in the documentation.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the Nix NVF configuration example in the README to use the nixpkgs-provided zen.nvim plugin package instead of a custom fetchFromGitHub + buildVimPlugin snippet.

Changes:

  • Replaces the README’s NVF plugin package example with the nixpkgs vimPlugins entry.
File summaries
File Description
README.md Simplifies the NVF Nix example by referencing the nixpkgs vim plugin package rather than an inline fetch/build expression.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md
@sand4rt
sand4rt merged commit 9ae3ce3 into main Aug 23, 2026
2 checks passed
@sand4rt
sand4rt deleted the docs-nix-package branch August 23, 2026 13:28
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