-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
vencord: 1.7.4 -> 1.7.8 #305032
vencord: 1.7.4 -> 1.7.8 #305032
Conversation
Signed-off-by: Sefa Eyeoglu <[email protected]>
Signed-off-by: Sefa Eyeoglu <[email protected]>
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
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.
Result of nixpkgs-review pr 305032
run on x86_64-linux 1
4 packages built:
- vencord
- vencord-web-extension
- vesktop
- webcord-vencord
could someone please merge this? |
should this be backported? |
I don't see a reason for not backporting. It's parent Discord is almost always backported to latest stable release. However this particular PR can't be automatically backported by the bot. One needs to manually cherry pick all the previous commits, including the file rename commit. Feel free to pick the job. |
See #305542 |
Description of changes
Closes #305030
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/
)Add a 👍 reaction to pull requests you find important.