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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
harmonist: init at 0.1 #62030
harmonist: init at 0.1 #62030
Conversation
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.
Plays fine. Saving and loading games also works.
pkgs/games/harmonist/default.nix
Outdated
management and character building, relying on items and player | ||
adaptability for character progression. | ||
''; | ||
homepage = https://harmonist.tuxfamily.org/; |
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.
Hint: We're going to deprecate unquoted URL syntax soon, if RFC 45 if approved.
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.
If you do the change from name to pname, you can update this as well.
@GrahamcOfBorg build harmonist |
pkgs/games/harmonist/default.nix
Outdated
|
||
buildGoPackage rec { | ||
|
||
name = "harmonist-${version}"; |
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.
Can be replaced by pname = "harmonist";
. Then name will be built by itself.
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.
Looks good overall, it would be great if you could look into the notes left by me and @JohnAZoidberg and then I have no problem with merging this.
@etu @JohnAZoidberg It's done. 馃懡 |
@freepotion Thanks! :) |
@etu Thank you too! :) |
Motivation for this change
To add a new roguelike game.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)馃懡