-
-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
rose-pine-cursor: init at 1.1.0 #287612
rose-pine-cursor: init at 1.1.0 #287612
Conversation
3fa45c6
to
d2221cb
Compare
2b0a2e3
to
92a193b
Compare
6f839f2
to
eb777f2
Compare
I by mistake did a force push deleting the commit that added the runhook, so I just redid it and moved it where it was |
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.
Please squash all your commits into one commit named rose-pine-cursor: init at 1.1.0
fix: meta Co-authored-by: Jeremy <jtbx@duck.com> fix: replace stdenv with stdenvNoCC Co-authored-by: Jeremy <jtbx@duck.com> style: fix indentation Co-authored-by: Jeremy <jtbx@duck.com> fix: replace r flag with R Co-authored-by: Jeremy <jtbx@duck.com> fix: variables fix: add closing parentheses at the end of the file feat: skip configure phase feat: skip build phase feat: add runhook for preInstall and postInstall in installPhase Co-authored-by: Jeremy <jtbx@duck.com> Revert "feat: skip configure phase" This reverts commit db36e57. Revert "feat: skip build phase" This reverts commit 92a193b.
eb777f2
to
d444fff
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.
Looks good to me :)
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.
Tested on my machine, completely functional.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1509 |
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Closes #287123
Add a 馃憤 reaction to pull requests you find important.