ci/github-script/commits: clarify comments
Splits the "occasionally" case into two, depending on whether the commit has a diff or was not cherry-picked at all. Prepares the next commit, where these are conditionally shown only.
This commit is contained in:
@@ -5,6 +5,11 @@ Some of the commits in this PR require the author's and reviewer's attention.
|
||||
Please follow the [backporting guidelines](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#how-to-backport-pull-requests) and cherry-pick with the `-x` flag.
|
||||
This requires changes to the unstable `master` and `staging` branches first, before backporting them.
|
||||
|
||||
Occasionally, it is not possible to cherry-pick exactly the same patch.
|
||||
This most frequently happens when resolving merge conflicts or when updating minor versions of packages which have already advanced to the next major on unstable.
|
||||
Occasionally, commits are not cherry-picked at all, for example when updating minor versions of packages which have already advanced to the next major on unstable.
|
||||
These commits can optionally be marked with a `Not-cherry-picked-because: <reason>` footer.
|
||||
|
||||
Sometimes it is not possible to cherry-pick exactly the same patch.
|
||||
This most frequently happens when resolving merge conflicts.
|
||||
The range-diff will help to review the resolution of conflicts.
|
||||
|
||||
If you need to merge this PR despite the warnings, please [dismiss](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/dismissing-a-pull-request-review) this review shortly before merging.
|
||||
|
||||
Reference in New Issue
Block a user