-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
spotube: 3.8.3 → 3.9.0 | add aarch64-linux support. #368631
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.
old -> new
should be used instead of old > new
according to the commit conventions for version updates.
Also, please put the maintainer list commit before the main commit.
one way to do this: git rebase -i HEAD~2
and swap the two commits
Please drop the unrelated commits |
@TomaSajt you're saying about these commits from others in others files? I don't know why is showing it, when I use git rebase -I just have one line uncommented wrote "noop".
I'm starting to use git and GitHub for PRs, so sorry if i'm doing something wrong. |
When doing
You will need to drop the 3 unneeded commits. you either have to remove the unneeded commits:
or you have to write
If you accidentally start rebasing more than what you need you can cancel the rebase by deleting all of the text. |
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 :)
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.