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
amp: init at 2019.06.11 #62978
amp: init at 2019.06.11 #62978
Conversation
21ba6f6
to
3f2e87a
Compare
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.
Quick review with a few small things I noticed.
3f2e87a
to
3eb3ca2
Compare
OK, except for the version number everything is resolved. |
ff2d808
to
c4efd1a
Compare
All done |
c4efd1a
to
d8810ec
Compare
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.
LGTM 👍
@GrahamcOfBorg build amp |
@sbourdeauducq Sorry I realized I missed something! You need to add |
AFAICT it should be rather platform-independent. It's just a regular Rust app... |
@sbourdeauducq I would say they should indeed, eg. |
d8810ec
to
8fb4a6a
Compare
Ok. |
@GrahamcOfBorg build amp |
@sbourdeauducq thank you very much for your contribution to nixpkgs! This looks like a really cool editor 🎉 |
Motivation for this change
Add a package for a nice text editor.
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)