New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NEW Athame-zsh #74804
NEW Athame-zsh #74804
Conversation
I recommend rereading CONTRIBUTING.md. Specifically the sections about commit messages and meta.description. This does not seem to respect the directory structure. Maybe move pkgs/shells/zsh/athame-zsh.nix to pkgs/shells/zsh/zsh-athame/default.nix athame and vimbed should probably be packaged separately. Do you actually need to override zsh? The other zsh addons do not seem to need to do this. Are you able to reference the patch you are applying in postPatch in the actual patch stage? Also are two of those lines in postPatch just comments? If so, then those should be removed. |
@@ -0,0 +1,39 @@ | |||
{stdenv, vim, zsh, ncurses, isNeovim ? false}: | |||
let athame = fetchGit { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use fetchFromGitHub
rev = "ded4200cb3312334d99ace794c4b308cd45f6b6e"; | ||
url = "https://github.com/ardagnir/athame"; | ||
}; | ||
vimbed = fetchGit { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use fetchFromGitHub
@@ -7670,4 +7670,9 @@ | |||
githubId = 52650; | |||
name = "Marc Busqué"; | |||
}; | |||
bananPasha = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use BananchickPasha
instead of bananPasha
to match the GitHub username
Thank you for your contributions.
|
Motivation for this change
There wasn't this package so i decided to add it
Things done
It works, but user has to create ~/.athamerc manually.
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @