strip PR template in merge commit message #102
Comments
3 tasks
karthikeyann
added a commit
to karthikeyann/ops-bot
that referenced
this issue
May 6, 2022
extracts text in "## description" section and skips text after "## checklist". Request from rapidsai#102
ajschmidt8
added a commit
that referenced
this issue
Jul 12, 2022
* extract description from PR body text extracts text in "## description" section and skips text after "## checklist". Request from #102 * fix this reference * add unit test for PR body text extraction * change pulls_get.json to pulls_get.ts * update auto_merger tests * formatting updates Co-authored-by: Karthikeyan Natarajan <[email protected]> Co-authored-by: AJ Schmidt <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
PR template with checklist and additional text helps authors to create better PRs. e.g. PR template is updated in PR rapidsai/cudf#10774
@gpucibot mergeuses PR description text as commit message verbatim. but this template text need not be present in commit message.Add some preprocessing to strip off PR template from PR description text before using it as commit message.
more discussion here rapidsai/cudf#10774 (comment) and https://nvidia.slack.com/archives/CALTSAKJ6/p1651590453510599?thread_ts=1651582344.289729&cid=CALTSAKJ6
The text was updated successfully, but these errors were encountered: