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
Paperwork update #81921
Paperwork update #81921
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.
can you clean up git history, there should only be 4 commits
use git rebase -i
It's impossible to use rebase after a merge. I can squash if you prefer. |
you can use |
although i almost never do merge commits, so maybe im wrong |
git rebase -i lists 11058 commits. I am afraid it is unworkable. I can either squash or leave the PR as is. Feel free to edit it to you liking, as I ticked the |
you can also do |
|
it's listed under https://nixos.org/nixpkgs/manual/#submitting-changes-making-patches that your git commits should be done in a way that "makes sense" (logical units). Usually to make it easy for cherry-picks to release branches. 3 bumps and 4 fixups to the paperwork package is not in the spirit of CONTRIBUTING.md |
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 groom git history, should be ~5 commits
c46044b
to
82b54d4
Compare
Motivation for this change
Things done
libinsane
pyinsane2
as it was only needed by older versions of paperwork and is now unmaintained upstreamsandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)