nix PR backporting script #26

Open
opened 2024-03-12 18:13:06 +00:00 by jade · 3 comments
Owner

it should:

  • cherry-pick with -x to indicate commit, and use -m1 for merge commits (.... also, somehow deal with rebase/squash-merged PRs)
  • add an Upstream-PR: https://x line
  • ensure there is a change-id without requiring that people tamper with their git hooks to have a (very scuffed) pre-commit-msg hook
  • Do the gh api whatevers necessary to make this happen.
  • Possibly import the PR body since it tends to be better than the average commit message, or at least just grab both of them and let the user delete the worse one.
it should: - cherry-pick with -x to indicate commit, and use `-m1` for merge commits (.... also, somehow deal with rebase/squash-merged PRs) - add an `Upstream-PR: https://x` line - ensure there is a change-id without requiring that people tamper with their git hooks to have a (very scuffed) pre-commit-msg hook - Do the `gh api` whatevers necessary to make this happen. - Possibly import the PR body since it tends to be better than the average commit message, or at least just grab both of them and let the user delete the worse one.
jade added the
devx
label 2024-03-12 18:13:06 +00:00
Owner

how many backports are you expecting going forward :dragneyes:

how many backports are you expecting going forward :dragneyes:
Author
Owner

Not that many, I just think this process is confusing and writing a script to give us nicer commit history would be good.

Not that many, I just think this process is confusing and writing a script to give us nicer commit history would be good.
Author
Owner
https://gerrit.lix.systems/c/lix/+/494
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lix-project/lix#26
No description provided.