-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
[Backport release-24.05] legcord: 1.0.1 -> 1.0.2 #350123
[Backport release-24.05] legcord: 1.0.1 -> 1.0.2 #350123
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.
nixpkgs-review
result
Generated using nixpkgs-review
.
Command: nixpkgs-review pr 350123
x86_64-linux
✅ 1 package built:
- legcord
I think you didn't actually use cherry-pick here, right? Would you mind cherry-picking the commits that you want to backport from the master branch, so that we have git include the cherry-pick information automatically? |
Correct, the source build from master has issues with electron on the release branch and armcord/legcord was always a binary build on 24.05. Given we are a month away from 24.11 release, my current thought is to leave it working as is and let the binary build be retired with 24.05. I am backporting the intent (legcord -> 1.0.2) rather than the actual source build in this case. |
@r-vdp Any objection to moving forward with the merge given the builds are actually different between |
https://github.com/Legcord/Legcord/releases/tag/v1.0.2
Result of
nixpkgs-review pr 350123
run on x86_64-linux 11 package built:
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.