Repository navigation
Conversation
|
Please try not to open and close 4 identical PRs in the space of 15m 👀 |
Sorry, I just have no other way to test these, I will find a way to though |
The documentation isn't very clear, how do I test it? |
Test what? It'll trigger CI on pushes/force-pushes. |
Your PR adds a dependency that isn't found in the version of Nixpkgs Nixvim is currently using. If you're really unfamiliar with git and github, you may wish to read some of the getting started docs 🙂 |
khaneliman
left a comment
There was a problem hiding this comment.
Current build checks fail. Module also needs the standard matching colorscheme test before merge.
| }; | ||
|
|
||
| plugins = { | ||
| autoload = defaultNullOpts.mkBool true ""; |
There was a problem hiding this comment.
issue (blocking): keep upstream settings freeform
These declarations mirror upstream settings and contain no useful descriptions. Replace settingsOptions with a representative settingsExample, as CONTRIBUTING.md recommends.
| wokerNM = { | ||
| email = "wokerNM@proton.me"; | ||
| github = "wokerNM"; | ||
| githubId = "215592475"; |
There was a problem hiding this comment.
| githubId = "215592475"; | |
| githubId = 215592475; |
issue (blocking): use numeric GitHub ID
Maintainer githubId values are integers. Remove quotes so maintainer metadata passes expected typing and rendering.
Added https://codeberg.org/ficd/ashen.nvim colorscheme, and myself to maintainer list