fix(docs): add a note about bundling and merge_pr_in_ci - #1165
Conversation
Fixes packit/packit#2733 Fixes PACKIT-5184 Signed-off-by: Matej Focko <mfocko@packit.dev>
|
🤖 Finished Review · ✅ Success · Started 8:15 AM UTC · Completed 8:28 AM UTC Commit: |
|
Build succeeded. ✔️ pre-commit SUCCESS in 1m 14s |
ReviewFindingsLow
Labels: PR adds documentation about the merge_pr_in_ci configuration key |
| RPM packages with an NVR referencing an unknown hash (the one created by the | ||
| Packit merge commit). | ||
|
|
||
| :::tip Packages with bundled dependencies |
There was a problem hiding this comment.
[low] scope-vs-intent
The linked issue (packit/packit#2733) reports a code bug in packit's prepare-sources path where git checkout main fails on vendored files. This PR closes the issue by documenting a workaround (merge_pr_in_ci: false) rather than fixing the code. Per maintainer analysis in the issue comments, the code fix approach was deliberately declined. Consider keeping the issue open or noting the resolution rationale before closing.
Suggested fix: Either keep the issue open and remove the Fixes closer from the PR body, or note the resolution rationale in the issue comments before closing.
Fixes packit/packit#2733
Fixes PACKIT-5184
Since there is no reply on the issue, I’m adding a note to the docs and closing down the issue.